Bug 16037 - Scraps not displayed in Orkut due to unclosed <iframe> tag
Summary: Scraps not displayed in Orkut due to unclosed <iframe> tag
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Evangelism (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL: http://www.orkut.com/Scrapbook.aspx
Keywords: GoogleBug, HasReduction, InRadar
Depends on:
Blocks:
 
Reported: 2007-11-18 05:17 PST by Indrek Siitan
Modified: 2009-01-06 11:21 PST (History)
4 users (show)

See Also:


Attachments
Test case (187 bytes, text/html)
2007-11-18 10:18 PST, David Kilzer (:ddkilzer)
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Indrek Siitan 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.
Comment 1 Matt Lilek 2007-11-18 09:57:51 PST
Confirmed with r27879.
Comment 2 David Kilzer (:ddkilzer) 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.

Comment 3 David Kilzer (:ddkilzer) 2007-11-18 10:18:05 PST
Created attachment 17366 [details]
Test case
Comment 4 David Storey 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.
Comment 5 Pereira 2007-11-20 11:39:26 PST
I'm a Googler and already contact Orkut Team to fix this.
Comment 6 David Kilzer (:ddkilzer) 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!

Comment 7 David Storey 2007-11-20 13:53:25 PST
One of my google contacts is looking for someone to fix it too.
Comment 8 David Kilzer (:ddkilzer) 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.

Comment 9 David Storey 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.

Comment 10 David Kilzer (:ddkilzer) 2008-02-03 06:26:14 PST
<rdar://problem/5721953>
Comment 11 Eric Seidel (no email) 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. :)
Comment 13 David Kilzer (:ddkilzer) 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.