Bug 36203

Summary: Add tests for "load shared worker in iframe"
Product: WebKit Reporter: Andrew Wilson <atwilson>
Component: WebCore JavaScriptAssignee: Andrew Wilson <atwilson>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch none

Andrew Wilson
Reported 2010-03-16 17:12:58 PDT
We found a bug in downstream Chromium related to launching shared workers simultaneously in a top-level document and in an iframe. Turns out that we don't explicitly test this case in any of the WebKit layout tests, so we should add a test.
Attachments
Patch (3.11 KB, patch)
2010-03-16 17:19 PDT, Andrew Wilson 		no flags
DetailsFormatted DiffDiff
Patch (3.13 KB, patch)
2010-03-16 17:37 PDT, Andrew Wilson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andrew Wilson
Comment 1 2010-03-16 17:19:45 PDT
Created attachment 50854 [details] Patch
David Levin
Comment 2 2010-03-16 17:34:29 PDT
Comment on attachment 50854 [details] Patch This looks good. Thanks! r- for mild issue having two files wit the same name "shared-worker-iframe.html" but having different purposes.
Andrew Wilson
Comment 3 2010-03-16 17:37:18 PDT
Created attachment 50857 [details] Patch
WebKit Commit Bot
Comment 4 2010-03-17 17:37:12 PDT
Comment on attachment 50857 [details] Patch Clearing flags on attachment: 50857 Committed r56136: <http://trac.webkit.org/changeset/56136>
WebKit Commit Bot
Comment 5 2010-03-17 17:37:16 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.