Bug 36203

Summary:

Add tests for "load shared worker in iframe"

Product:

WebKit

Reporter:

Andrew Wilson <atwilson>

Component:

WebCore JavaScript

Assignee:

Andrew Wilson <atwilson>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

OS X 10.5

Attachments:

Description	Flags
Patch	none
Patch	none

Description Andrew Wilson 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.

Comment 1 Andrew Wilson 2010-03-16 17:19:45 PDT

Created attachment 50854 [details]
Patch

Comment 2 David Levin 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.

Comment 3 Andrew Wilson 2010-03-16 17:37:18 PDT

Created attachment 50857 [details]
Patch

Comment 4 WebKit Commit Bot 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>

Comment 5 WebKit Commit Bot 2010-03-17 17:37:16 PDT

All reviewed patches have been landed.  Closing bug.