WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 104704
85768
[Chromium] Move sharedWorkerRepository to Platform.h
https://bugs.webkit.org/show_bug.cgi?id=85768
Summary
[Chromium] Move sharedWorkerRepository to Platform.h
Mark Pilgrim (Google)
Reported
2012-05-06 20:11:58 PDT
[Chromium] Move sharedWorkerRepository to Platform.h
Attachments
Patch
(6.54 KB, patch)
2012-05-06 20:12 PDT
,
Mark Pilgrim (Google)
abarth
: review-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1
2012-05-06 20:12:38 PDT
Created
attachment 140463
[details]
Patch
WebKit Review Bot
Comment 2
2012-05-06 20:14:04 PDT
Please wait for approval from
abarth@webkit.org
,
dglazkov@chromium.org
,
fishd@chromium.org
,
jamesr@chromium.org
or
tkent@chromium.org
before submitting, as this patch contains changes to the Chromium public API. See also
https://trac.webkit.org/wiki/ChromiumWebKitAPI
.
Adam Barth
Comment 3
2012-05-06 20:49:27 PDT
Comment on
attachment 140463
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=140463&action=review
> Source/WebKit/chromium/public/platform/WebKitPlatformSupport.h:-63 > -class WebSharedWorkerRepository; // FIXME: Does this belong in platform?
This one is tricky because WebSharedWorkerRepository depends on WebSharedWorker, which depends on a bunch of "non-platform" concepts (e.g., ScriptExecutionContext, ContentSecurityPolicy). It's likely this will need to move outside of the platform part of the API... We'd need to look more closely at where.
Mark Pilgrim (Google)
Comment 4
2012-12-11 12:54:47 PST
*** This bug has been marked as a duplicate of
bug 104704
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug