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

Side by Side Diff: content/browser/frame_host/render_frame_host_manager.cc

Issue 2557063002: Upgrade Insecure Requests: bugfixes, tests, and support for OOPIF.
Patch Set: Addressed comments (@nasko #2). Created 4 years 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 unified diff | Download patch
« no previous file with comments | « content/browser/frame_host/render_frame_host_manager.h ('k') | content/common/frame_messages.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/frame_host/render_frame_host_manager.h" 5 #include "content/browser/frame_host/render_frame_host_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <string> 10 #include <string>
(...skipping 964 matching lines...) Expand 10 before | Expand all | Expand 10 after
975 blink::WebInsecureRequestPolicy policy) { 975 blink::WebInsecureRequestPolicy policy) {
976 if (!SiteIsolationPolicy::AreCrossProcessFramesPossible()) 976 if (!SiteIsolationPolicy::AreCrossProcessFramesPossible())
977 return; 977 return;
978 978
979 for (const auto& pair : proxy_hosts_) { 979 for (const auto& pair : proxy_hosts_) {
980 pair.second->Send(new FrameMsg_EnforceInsecureRequestPolicy( 980 pair.second->Send(new FrameMsg_EnforceInsecureRequestPolicy(
981 pair.second->GetRoutingID(), policy)); 981 pair.second->GetRoutingID(), policy));
982 } 982 }
983 } 983 }
984 984
985 void RenderFrameHostManager::OnEnforceInsecureNavigationsSet(
986 const std::vector<uint32_t>& insecure_navigations_set) {
987 if (!SiteIsolationPolicy::AreCrossProcessFramesPossible())
988 return;
989
990 for (const auto& pair : proxy_hosts_) {
991 pair.second->Send(new FrameMsg_EnforceInsecureNavigationsSet(
992 pair.second->GetRoutingID(), insecure_navigations_set));
993 }
994 }
995
985 void RenderFrameHostManager::OnDidUpdateFrameOwnerProperties( 996 void RenderFrameHostManager::OnDidUpdateFrameOwnerProperties(
986 const FrameOwnerProperties& properties) { 997 const FrameOwnerProperties& properties) {
987 if (!SiteIsolationPolicy::AreCrossProcessFramesPossible()) 998 if (!SiteIsolationPolicy::AreCrossProcessFramesPossible())
988 return; 999 return;
989 1000
990 // FrameOwnerProperties exist only for frames that have a parent. 1001 // FrameOwnerProperties exist only for frames that have a parent.
991 CHECK(frame_tree_node_->parent()); 1002 CHECK(frame_tree_node_->parent());
992 SiteInstance* parent_instance = 1003 SiteInstance* parent_instance =
993 frame_tree_node_->parent()->current_frame_host()->GetSiteInstance(); 1004 frame_tree_node_->parent()->current_frame_host()->GetSiteInstance();
994 1005
(...skipping 1741 matching lines...) Expand 10 before | Expand all | Expand 10 after
2736 resolved_url)) { 2747 resolved_url)) {
2737 DCHECK(!dest_instance || 2748 DCHECK(!dest_instance ||
2738 dest_instance == render_frame_host_->GetSiteInstance()); 2749 dest_instance == render_frame_host_->GetSiteInstance());
2739 return false; 2750 return false;
2740 } 2751 }
2741 2752
2742 return true; 2753 return true;
2743 } 2754 }
2744 2755
2745 } // namespace content 2756 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_frame_host_manager.h ('k') | content/common/frame_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698