Summary: | Threaded HTML parser should pass fast/parser/iframe-sets-parent-to-javascript-url.html | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||||
Component: | New Bugs | Assignee: | Adam Barth <abarth> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric, esprehn+autocc, japhet, ojan.autocc, tonyg, vsevik, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 110724 | ||||||||
Bug Blocks: | 106127 | ||||||||
Attachments: |
|
Description
Adam Barth
2013-02-22 13:30:25 PST
Created attachment 189815 [details]
Patch
Comment on attachment 189815 [details]
Patch
This is fine. We could be more sophisticated and just handle the first chunk on the main thread. :)
That might be worth doing in general for loads since we'll get to the first byte faster. Comment on attachment 189815 [details] Patch Clearing flags on attachment: 189815 Committed r143797: <http://trac.webkit.org/changeset/143797> All reviewed patches have been landed. Closing bug. Re-opened since this is blocked by bug 110724 Downstream chromium content_browsertests BookmarkletTest.NonEmptyResult test was also broken by this patch. http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=content_browsertests&tests=BookmarkletTest.NonEmptyResult Created attachment 190102 [details]
Patch for landing
Comment on attachment 190102 [details] Patch for landing Clearing flags on attachment: 190102 Committed r144000: <http://trac.webkit.org/changeset/144000> All reviewed patches have been landed. Closing bug. |