Bug 16230 - document.write causes the links to appear twice (noscript content is not ignored)
Summary: document.write causes the links to appear twice (noscript content is not igno...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://xj.9you.com/pic_02.html
Keywords:
Depends on: 8961
Blocks:
  Show dependency tree / graph
 
Reported: 2007-12-01 21:37 PST by Anantha Keesara
Modified: 2010-08-16 02:10 PDT (History)
3 users (show)

See Also:

Attachments
Screenshot (103.66 KB, image/jpeg)
2007-12-01 21:39 PST, Anantha Keesara 		no flags Details
further reduced test case (973 bytes, application/zip)
2007-12-03 01:46 PST, Alexey Proskuryakov 		no flags Details
Patch (1.69 KB, patch)
2010-08-16 00:35 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
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 Anantha Keesara 2007-12-01 21:37:50 PST 
Reduction:
http://help.improve.safari.googlepages.com/xj-9you.html

Verified against webkit nightly: r28278
Comment 1 Anantha Keesara 2007-12-01 21:39:13 PST 
Created attachment 17638 [details]
Screenshot
Comment 2 Alexey Proskuryakov 2007-12-03 01:46:30 PST 
Created attachment 17669 [details]
further reduced test case

Somehow, we fail to ignore <noscript> content here.
Comment 3 Eric Seidel (no email) 2008-01-17 04:19:14 PST 
This has even stranger behavior when run locally (for me).  If I dump the contents of the .js file into a <script> tag in the html file, then all I see is "); in the rendered document.

I wonder if this is related to bug 8961.  Marking it as depends on, in case it is.
Comment 4 Adam Barth 2010-08-15 00:13:10 PDT 
This appears to have been fixed by the HTML5 parser.

@Eric: that's because the </script> in the JavaScript string closes the script block.
Comment 5 Adam Barth 2010-08-16 00:27:00 PDT 
ap...  always wanting more tests :)
Comment 6 Adam Barth 2010-08-16 00:35:46 PDT 
Created attachment 64474 [details]
Patch
Comment 7 Alexey Proskuryakov 2010-08-16 00:45:19 PDT 
Comment on attachment 64474 [details]
Patch

Tests!!!
Comment 8 WebKit Commit Bot 2010-08-16 02:10:19 PDT 
Comment on attachment 64474 [details]
Patch

Clearing flags on attachment: 64474

Committed r65402: <http://trac.webkit.org/changeset/65402>
Comment 9 WebKit Commit Bot 2010-08-16 02:10:25 PDT 
All reviewed patches have been landed.  Closing bug.