Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(560)

Unified Diff: third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp

Issue 2733283004: Expose WebServiceWorkerNetworkProvider on DataSource (Closed)
Patch Set: consistency Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp b/third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp
index 3135c41469973049a984e8b5e991e2205798430b..7ed96708c3c7bc9afbc34b0f7d0a3c515733534e 100644
--- a/third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp
+++ b/third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp
@@ -59,6 +59,7 @@
#include "platform/weborigin/SecurityOrigin.h"
#include "public/platform/Platform.h"
#include "public/platform/WebURLRequest.h"
+#include "public/platform/modules/serviceworker/WebServiceWorkerNetworkProvider.h"
#include "public/platform/modules/serviceworker/WebServiceWorkerProvider.h"
#include "public/web/WebConsoleMessage.h"
#include "public/web/WebDevToolsAgent.h"
@@ -66,7 +67,6 @@
#include "public/web/WebView.h"
#include "public/web/WebWorkerContentSettingsClientProxy.h"
#include "public/web/modules/serviceworker/WebServiceWorkerContextClient.h"
-#include "public/web/modules/serviceworker/WebServiceWorkerNetworkProvider.h"
#include "web/IndexedDBClientImpl.h"
#include "web/ServiceWorkerGlobalScopeClientImpl.h"
#include "web/ServiceWorkerGlobalScopeProxy.h"
@@ -333,21 +333,21 @@ void WebEmbeddedWorkerImpl::loadShadowPage() {
void WebEmbeddedWorkerImpl::willSendRequest(WebLocalFrame* frame,
WebURLRequest& request) {
- if (m_networkProvider)
- m_networkProvider->willSendRequest(frame->dataSource(), request);
+ auto* networkProvider =
+ frame->dataSource()->getServiceWorkerNetworkProvider();
+ if (networkProvider)
+ networkProvider->willSendRequest(request);
}
void WebEmbeddedWorkerImpl::didFinishDocumentLoad(WebLocalFrame* frame) {
DCHECK(!m_mainScriptLoader);
- DCHECK(!m_networkProvider);
DCHECK(m_mainFrame);
DCHECK(m_workerContextClient);
DCHECK(m_loadingShadowPage);
DCHECK(!m_askedToTerminate);
m_loadingShadowPage = false;
- m_networkProvider =
- WTF::wrapUnique(m_workerContextClient->createServiceWorkerNetworkProvider(
- frame->dataSource()));
+ frame->dataSource()->setServiceWorkerNetworkProvider(WTF::wrapUnique(
+ m_workerContextClient->createServiceWorkerNetworkProvider()));
m_mainScriptLoader = WorkerScriptLoader::create();
m_mainScriptLoader->setRequestContext(
WebURLRequest::RequestContextServiceWorker);
« no previous file with comments | « third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.h ('k') | third_party/WebKit/Source/web/WebEmbeddedWorkerImplTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698