Summary: | ServiceWorkerContextManager should be an IPC::MessageReceiver | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||||
Component: | WebKit2 | Assignee: | Chris Dumez <cdumez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | beidson, ggaren, rniwa, webkit-bug-importer, youennf | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 178534 | ||||||||||
Attachments: |
|
Description
Chris Dumez
2017-10-19 16:42:25 PDT
Created attachment 324315 [details]
Patch
Created attachment 324316 [details]
Patch
Comment on attachment 324316 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=324316&action=review > Source/WebKit/WebProcess/Storage/ServiceWorkerContextManager.messages.in:27 > + StartFetchInServiceWorker(uint64_t serverConnectionIdentifier, uint64_t fetchIdentifier, uint64_t serviceWorkerIdentifier, WebCore::ResourceRequest request, struct WebCore::FetchOptions options) Can we rename these to StartWorker and Fetch since they are now scoped by ServiceWorkerContextManager? > Source/WebKit/WebProcess/WebProcess.cpp:665 > #if ENABLE(SERVICE_WORKER) probably > Source/WebKit/WebProcess/WebProcess.cpp:667 > + if (m_serviceWorkerManager) Can you add ASSERT(m_serviceWorkerManager); Created attachment 324327 [details]
Patch
Comment on attachment 324327 [details] Patch Clearing flags on attachment: 324327 Committed r223739: <https://trac.webkit.org/changeset/223739> All reviewed patches have been landed. Closing bug. |