RESOLVED FIXED 16037
Scraps not displayed in Orkut due to unclosed <iframe> tag
https://bugs.webkit.org/show_bug.cgi?id=16037
Summary Scraps not displayed in Orkut due to unclosed <iframe> tag
Indrek Siitan
Reported 2007-11-18 05:17:30 PST
As the summary says - the "scraps" in Orkut are not displayed at all - they appear in the source, but when rendering, the page is cut off after the title bar for the module.
Attachments
Test case (187 bytes, text/html)
2007-11-18 10:18 PST, David Kilzer (:ddkilzer)
no flags
Matt Lilek
Comment 1 2007-11-18 09:57:51 PST
Confirmed with r27879.
David Kilzer (:ddkilzer)
Comment 2 2007-11-18 10:17:17 PST
This issue is caused by an unclosed <iframe> tag in a document with an HTML 4.01 Transitional doctype: <div class="floatdiv" id="iframe_parent"> <iframe id="scrap_extn_iframe" name="scrap_extn_iframe" src=""/> <!--[if lte IE 7.0]><iframe></iframe><![endif]--> </div> Both Safari 2, Safari 3, ToT WebKit and Opera 9.24/9.50b leave the rest of the page blank because their parsers are looking for an </iframe> closing tag. Firefox 2.0.0.9 and 3.0b2pre (Nov 18, 2007) both apparently find the </iframe> within the comment (per the coloration of their View Source window), which is highly suspect as well, but makes the page work for Firefox. MSIE 7 apparently recognizes the self-closing <iframe/> tag (even though the document isn't XHTML), and of course parses the an additional <iframe></iframe> tags inside the VBScript. Best fix would be to have Google fix the self-closing iframe to: <iframe id="scrap_extn_iframe" name="scrap_extn_iframe" src=""></iframe> Marking this as an evangelism bug due to the above analysis.
David Kilzer (:ddkilzer)
Comment 3 2007-11-18 10:18:05 PST
Created attachment 17366 [details] Test case
David Storey
Comment 4 2007-11-18 10:55:47 PST
We (Opera) have the same issue in out bts. We agree it is a evangelism issue and FF are incorrect. I'm activly seeking a Google Orkut contact to fix this.
Pereira
Comment 5 2007-11-20 11:39:26 PST
I'm a Googler and already contact Orkut Team to fix this.
David Kilzer (:ddkilzer)
Comment 6 2007-11-20 12:53:16 PST
(In reply to comment #5) > I'm a Googler and already contact Orkut Team to fix this. Thank you!
David Storey
Comment 7 2007-11-20 13:53:25 PST
One of my google contacts is looking for someone to fix it too.
David Kilzer (:ddkilzer)
Comment 8 2007-11-20 15:18:09 PST
(In reply to comment #7) > One of my google contacts is looking for someone to fix it too. Thanks, David! I also cc-ed abob@google.com for this reason, although he's been quiet for a while now.
David Storey
Comment 9 2007-11-20 15:28:41 PST
I have: "I talked to the product manager for Orkut and we have produced a fix for the issue which should be deployed in the next production push for Orkut (late November, early December). Please let me know if you have any further questions. " We are getting tons of dupes on this (160 a day for Mini only). They offered to do some sort of pre canned response in the mean time.
David Kilzer (:ddkilzer)
Comment 10 2008-02-03 06:26:14 PST
Eric Seidel (no email)
Comment 11 2008-05-08 11:25:38 PDT
So if this has been resolved on Orkut's side, the only thing left to decide is if this is proper HTML5 parsing or not. If this is correct HTML5 parsing, then we can close this bug, if not, we'll need to fix it. :)
David Kilzer (:ddkilzer)
Comment 13 2009-01-06 11:21:47 PST
Actually(In reply to comment #12) > Closing as invalid. Actually, this bug was about an evangelism issue in Orkut (a Google property). That bug seems to be fixed now, so I'm going to reopen this to reclose it as RESOLVED FIXED.
Note You need to log in before you can comment on or make changes to this bug.