Bug 16230

Summary:

document.write causes the links to appear twice (noscript content is not ignored)

Product:

WebKit

Reporter:

Anantha Keesara <anantha>

Component:

DOM

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, ap, commit-queue

Priority:

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	Details Formatted Diff Diff
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.