Bug 16230

Summary: document.write causes the links to appear twice (noscript content is not ignored)
Product: WebKit Reporter: Anantha Keesara <anantha>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ap, commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://xj.9you.com/pic_02.html
Bug Depends on: 8961    
Bug Blocks:    
Attachments:
Description Flags
Screenshot
none
further reduced test case
none
Patch none

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.