Closed
Bug 652333
Opened 14 years ago
Closed 12 years ago
Google Gadget frame not displaying
Categories
(Tech Evangelism Graveyard :: English Other, defect)
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.
Comment 1•14 years ago
|
||
*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
Comment 2•14 years ago
|
||
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
Updated•10 years ago
|
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•