Closed Bug 652333 Opened 14 years ago Closed 12 years ago

Google Gadget frame not displaying

Categories

(Tech Evangelism Graveyard :: English Other, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: cosmo.guffa, Unassigned)

References

()

Details

User-Agent: Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0 Build Identifier: Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0 Javascript photo gallery to display Picasa photos contained within a Google Gadget is not displaying any content as it had in Firefox 3.6. Reproducible: Always Steps to Reproduce: 1. Visit site Actual Results: Frame is empty Expected Results: A photo gallery similar to the picture above it should be displayed in the frame. Paged worked in last stable version of Firefox 3.6 in both Linux and as a wine-emulated Windows version.
*This also happens on Google Chrome12.0.742.5 canary build. *Disabled HTML5 helps.user_pref("html5.enable", false);
Component: Layout: HTML Frames → HTML: Parser
OS: Linux → All
QA Contact: layout.html-frames → parser
Version: unspecified → 2.0 Branch
Oops s/html5.enable/html5.parser.enable/
The page works in Opera Ragnarök and in Chromium 10, so this isn't a typical parsing algorithm issue.
It appears that the site uses a pointlessly complex script loading pattern that isn't supposed to load scripts per HTML5. From http://js.hohli.com/photogallery/picasa.gallery.js : function loadJS(href) { with (document) { var span = createElement('SPAN'); span.style.display = 'none'; body.insertBefore(span, body.lastChild); span.innerHTML = 'Text for stupid IE.<s'+'cript></' + 'script>'; setTimeout(function() { var s = span.getElementsByTagName('script')[0]; s.language = 'JavaScript'; if (s.setAttribute) s.setAttribute('src', href); else s.src = href; }, 10); }} When the script element is created using innerHTML, it is marked as "already executed", so the script won't execute when its src attribute is set. Note that this behavior is brittle in IE, so the "Text for stupid IE." bit manages to make IE not set the "already executed" flag.
Assignee: nobody → english-other
Status: UNCONFIRMED → NEW
Component: HTML: Parser → English Other
Ever confirmed: true
Product: Core → Tech Evangelism
QA Contact: parser → english-other
Version: 2.0 Branch → unspecified
The site now loads http://photogallery.hohli.com/scripts/picasa.gallery.js that doesn't have the offending loadJS() function anymore.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.