Bug 110529

Summary:

Threaded HTML Parser fails fast/dom/Document/readystate.html

Product:

WebKit

Reporter:

Adam Barth <abarth>

Component:

WebCore Misc.

Assignee:

Adam Barth <abarth>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

eric, esprehn+autocc, ojan.autocc, tonyg, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

106127

Attachments:

Description	Flags
Patch	none
Patch for landing	none

Adam Barth

Reported 2013-02-21 16:28:52 PST

Reduction: <body> <iframe></iframe> <script> testRunner.dumpAsText(); var doc = frames[0].document; doc.open(); doc.close(); console.log(doc.body); </script> This outputs "null" with the threaded parser and non-null without.

Attachments
Patch (3.33 KB, patch) 2013-02-21 16:49 PST, Adam Barth	no flags	Details Formatted Diff Diff
Patch for landing (3.37 KB, patch) 2013-02-21 16:54 PST, Adam Barth	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Eric Seidel (no email)

Comment 1 2013-02-21 16:30:59 PST

Just have to make sure you pump at least once for the body to be created. Presumably for this script created parser we are failing to pump during flush, etc. We should have had code for this?

Adam Barth

Comment 2 2013-02-21 16:49:32 PST

Created attachment 189638 [details] Patch

Eric Seidel (no email)

Comment 3 2013-02-21 16:51:18 PST

Comment on attachment 189638 [details] Patch You could have had it spit out "PASS" instead. :) But LGTM.

Adam Barth

Comment 4 2013-02-21 16:51:48 PST

Comment on attachment 189638 [details] Patch Alright

Adam Barth

Comment 5 2013-02-21 16:54:21 PST

Created attachment 189640 [details] Patch for landing

WebKit Review Bot

Comment 6 2013-02-21 17:15:12 PST

Comment on attachment 189640 [details] Patch for landing Clearing flags on attachment: 189640 Committed r143670: <http://trac.webkit.org/changeset/143670>

WebKit Review Bot

Comment 7 2013-02-21 17:15:15 PST

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.