Bug 85768

Summary: [Chromium] Move sharedWorkerRepository to Platform.h
Product: WebKit Reporter: Mark Pilgrim (Google) <pilgrim>
Component: WebKit Misc.Assignee: Mark Pilgrim (Google) <pilgrim>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: abarth, dglazkov, fishd, haraken, jamesr, tkent+wkapi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 95861    
Bug Blocks: 82948    
Attachments:
Description Flags
Patch abarth: review-

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-
DetailsFormatted DiffDiff
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.