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

Unified Diff: content/renderer/render_frame_impl.cc

Issue 1191293002: Don't create ServiceWorkerProviderHost for sandboxed frames without allow-same-origin flag. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 9c7635523fe8714489d7c7aafd50e2f96a924997..69767a442e64516b272ecd6200b9504c98e520da 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -2460,9 +2460,14 @@ void RenderFrameImpl::didCreateDataSource(blink::WebLocalFrame* frame,
// exists).
if (!ServiceWorkerNetworkProvider::FromDocumentState(
DocumentState::FromDataSource(datasource))) {
+ ServiceWorkerProviderType provider_type =
+ SERVICE_WORKER_PROVIDER_FOR_WINDOW;
+ if ((frame->effectiveSandboxFlags() & blink::WebSandboxFlags::Origin) ==
+ blink::WebSandboxFlags::Origin) {
+ provider_type = SERVICE_WORKER_PROVIDER_FOR_SANDBOXED_FRAME;
+ }
scoped_ptr<ServiceWorkerNetworkProvider> network_provider(
- new ServiceWorkerNetworkProvider(routing_id_,
- SERVICE_WORKER_PROVIDER_FOR_WINDOW));
+ new ServiceWorkerNetworkProvider(routing_id_, provider_type));
ServiceWorkerNetworkProvider::AttachToDocumentState(
DocumentState::FromDataSource(datasource),
network_provider.Pass());
@@ -3669,6 +3674,8 @@ bool RenderFrameImpl::isControlledByServiceWorker(WebDataSource& data_source) {
ServiceWorkerNetworkProvider* provider =
ServiceWorkerNetworkProvider::FromDocumentState(
DocumentState::FromDataSource(&data_source));
+ if (!provider->context())
+ return false;
return provider->context()->controller_handle_id() !=
kInvalidServiceWorkerHandleId;
}
@@ -3677,8 +3684,7 @@ int64_t RenderFrameImpl::serviceWorkerID(WebDataSource& data_source) {
ServiceWorkerNetworkProvider* provider =
ServiceWorkerNetworkProvider::FromDocumentState(
DocumentState::FromDataSource(&data_source));
-
- if (provider->context()->controller())
+ if (provider->context() && provider->context()->controller())
return provider->context()->controller()->version_id();
return kInvalidServiceWorkerVersionId;
}

Powered by Google App Engine
This is Rietveld 408576698