Index: content/browser/frame_host/navigation_request.cc |
diff --git a/content/browser/frame_host/navigation_request.cc b/content/browser/frame_host/navigation_request.cc |
index b8ef2b115fbab3577b2e5ba6512eb91817b09bde..17e42682553082f6df284c6416f8545bba084b4e 100644 |
--- a/content/browser/frame_host/navigation_request.cc |
+++ b/content/browser/frame_host/navigation_request.cc |
@@ -301,6 +301,7 @@ void NavigationRequest::OnResponseStarted( |
} |
// Update the service worker params of the request params. |
+ // TODO(falken): This should do an isSecureContext check too. |
jww
2016/05/24 18:06:03
To ask a dumb question, why not add that check rig
falken
2016/05/25 09:19:58
I didn't look too hard, but there's no WebFrame he
clamy
2016/05/25 11:21:46
If you're checking for the sandbox flags, you can
falken
2016/05/26 02:09:49
I want to do an ancestor walk over the frame tree,
clamy
2016/05/26 15:45:31
FrameTreeNodes do allow you to walk over the Frame
alexmos
2016/05/26 22:13:35
Yeah, you should be able to follow the FrameTreeNo
falken
2016/05/27 05:47:03
Sorry yes I meant "FrameTreeNode doesn't have the
clamy
2016/05/30 16:22:42
Probably. In any case, we'll have to make signific
|
request_params_.should_create_service_worker = |
(frame_tree_node_->pending_sandbox_flags() & |
blink::WebSandboxFlags::Origin) != blink::WebSandboxFlags::Origin; |