RESOLVED FIXED Bug 16230
document.write causes the links to appear twice (noscript content is not ignored) 
https://bugs.webkit.org/show_bug.cgi?id=16230
Summary document.write causes the links to appear twice (noscript content is not igno...
Anantha Keesara
Reported 2007-12-01 21:37:50 PST
Reduction: http://help.improve.safari.googlepages.com/xj-9you.html Verified against webkit nightly: r28278
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
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anantha Keesara
Comment 1 2007-12-01 21:39:13 PST
Created attachment 17638 [details] Screenshot
Alexey Proskuryakov
Comment 2 2007-12-03 01:46:30 PST
Created attachment 17669 [details] further reduced test case Somehow, we fail to ignore <noscript> content here.
Eric Seidel (no email)
Comment 3 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.
Adam Barth
Comment 4 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.
Adam Barth
Comment 5 2010-08-16 00:27:00 PDT
ap... always wanting more tests :)
Adam Barth
Comment 6 2010-08-16 00:35:46 PDT
Created attachment 64474 [details] Patch
Alexey Proskuryakov
Comment 7 2010-08-16 00:45:19 PDT
Comment on attachment 64474 [details] Patch Tests!!!
WebKit Commit Bot
Comment 8 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>
WebKit Commit Bot
Comment 9 2010-08-16 02:10:25 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.