Bug 16722 - Page rendered blank due to difference in handling of HTML comment inside script element
Summary: Page rendered blank due to difference in handling of HTML comment inside scri...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 523.x (Safari 3)
Hardware: All All
: P1 Normal
Assignee: Nobody
URL:
Keywords: HasReduction, InRadar
: 12009 25636 28838 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-01-03 07:34 PST by jonathanjohnsson
Modified: 2011-03-30 00:28 PDT (History)
9 users (show)

See Also:

Attachments
Reduced test case (214 bytes, text/html)
2008-01-03 08:07 PST, Mark Rowe (bdash) 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description jonathanjohnsson 2008-01-03 07:34:31 PST 
Going to http://www.skype.com/allfeatures/callphones/ shows a blank page. Firefox shows the page as expected. The same happens both in released Safari 3.04 and in nightly r29120.
Comment 1 Mark Rowe (bdash) 2008-01-03 08:07:58 PST 
Created attachment 18251 [details]
Reduced test case

The erroneous behaviour is triggered by the following snippet:
<script type="text/javascript">
//<!--[CDATA[
//]]>
</script>

<!-- Extra comment -->


Firefox sees the initial comment as ending inside the script element, while WebKit treats it as extending past "Extra comment".
Comment 2 Mark Rowe (bdash) 2008-01-03 08:09:13 PST 
<rdar://problem/5668296>
Comment 3 Robert Blaut 2008-02-18 00:51:31 PST 
*** Bug 12009 has been marked as a duplicate of this bug. ***
Comment 4 Robert Blaut 2008-02-18 00:59:52 PST 
I updated URL since previous site was updated and the problem is not visible on that page any more.
Comment 5 Robert Blaut 2008-02-27 05:17:41 PST 
The site has no longer the issue visible.
Comment 6 Alexey Proskuryakov 2009-05-08 10:07:09 PDT 
*** Bug 25636 has been marked as a duplicate of this bug. ***
Comment 7 Alexey Proskuryakov 2009-05-08 10:07:53 PDT 
According to html5lib, WebKit does what the spec says here.
Comment 8 Alexey Proskuryakov 2009-09-01 10:36:23 PDT 
*** Bug 28838 has been marked as a duplicate of this bug. ***
Comment 9 Jerome Leclanche 2009-09-01 11:30:32 PDT 
(In reply to comment #7)
> According to html5lib, WebKit does what the spec says here.

"View source" doesn't actually show the entire source. While it may be intended, the parser dying should be fixed.
Comment 10 Alexey Proskuryakov 2009-09-01 12:36:09 PDT 
1. Safari "View Source" does show the entire source.
2. What do you mean by "parser dying"? A crash?

Since we do what HTML5 tells us to do, the most direct route forward is to discuss this issue with WhatWG or W3C HTML5 work groups.
Comment 11 Adam Barth 2010-09-20 02:31:18 PDT 
Fixed by the HTML5 parser.