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

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

Issue 2869423002: PlzNavigate: Do not disclose urls between cross-origin renderers. (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
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_impl.h" 5 #include "content/browser/frame_host/render_frame_host_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 #include "mojo/public/cpp/bindings/strong_binding.h" 112 #include "mojo/public/cpp/bindings/strong_binding.h"
113 #include "mojo/public/cpp/system/data_pipe.h" 113 #include "mojo/public/cpp/system/data_pipe.h"
114 #include "services/service_manager/public/cpp/connector.h" 114 #include "services/service_manager/public/cpp/connector.h"
115 #include "services/service_manager/public/cpp/interface_provider.h" 115 #include "services/service_manager/public/cpp/interface_provider.h"
116 #include "third_party/WebKit/public/platform/WebFeaturePolicy.h" 116 #include "third_party/WebKit/public/platform/WebFeaturePolicy.h"
117 #include "ui/accessibility/ax_tree.h" 117 #include "ui/accessibility/ax_tree.h"
118 #include "ui/accessibility/ax_tree_id_registry.h" 118 #include "ui/accessibility/ax_tree_id_registry.h"
119 #include "ui/accessibility/ax_tree_update.h" 119 #include "ui/accessibility/ax_tree_update.h"
120 #include "ui/gfx/geometry/quad_f.h" 120 #include "ui/gfx/geometry/quad_f.h"
121 #include "url/gurl.h" 121 #include "url/gurl.h"
122 #include "url/origin.h"
122 123
123 #if defined(OS_ANDROID) 124 #if defined(OS_ANDROID)
124 #include "content/browser/android/java_interfaces_impl.h" 125 #include "content/browser/android/java_interfaces_impl.h"
125 #include "content/browser/frame_host/render_frame_host_android.h" 126 #include "content/browser/frame_host/render_frame_host_android.h"
126 #include "content/browser/media/android/media_player_renderer.h" 127 #include "content/browser/media/android/media_player_renderer.h"
127 #include "content/public/browser/android/java_interfaces.h" 128 #include "content/public/browser/android/java_interfaces.h"
128 #include "media/base/audio_renderer_sink.h" 129 #include "media/base/audio_renderer_sink.h"
129 #include "media/base/video_renderer_sink.h" 130 #include "media/base/video_renderer_sink.h"
130 #include "media/mojo/services/mojo_renderer_service.h" // nogncheck 131 #include "media/mojo/services/mojo_renderer_service.h" // nogncheck
131 #endif 132 #endif
(...skipping 834 matching lines...) Expand 10 before | Expand all | Expand 10 after
966 // process should be ignored until the next commit. 967 // process should be ignored until the next commit.
967 set_nav_entry_id(0); 968 set_nav_entry_id(0);
968 } 969 }
969 970
970 void RenderFrameHostImpl::ReportContentSecurityPolicyViolation( 971 void RenderFrameHostImpl::ReportContentSecurityPolicyViolation(
971 const CSPViolationParams& violation_params) { 972 const CSPViolationParams& violation_params) {
972 Send(new FrameMsg_ReportContentSecurityPolicyViolation(routing_id_, 973 Send(new FrameMsg_ReportContentSecurityPolicyViolation(routing_id_,
973 violation_params)); 974 violation_params));
974 } 975 }
975 976
977 bool RenderFrameHostImpl::IsOriginSafeToUseInCspViolation(
978 const url::Origin& origin) const {
979 return origin.IsSameOriginWith(last_committed_origin_);
980 }
981
976 bool RenderFrameHostImpl::SchemeShouldBypassCSP( 982 bool RenderFrameHostImpl::SchemeShouldBypassCSP(
977 const base::StringPiece& scheme) { 983 const base::StringPiece& scheme) {
978 // Blink uses its SchemeRegistry to check if a scheme should be bypassed. 984 // Blink uses its SchemeRegistry to check if a scheme should be bypassed.
979 // It can't be used on the browser process. It is used for two things: 985 // It can't be used on the browser process. It is used for two things:
980 // 1) Bypassing the "chrome-extension" scheme when chrome is built with the 986 // 1) Bypassing the "chrome-extension" scheme when chrome is built with the
981 // extensions support. 987 // extensions support.
982 // 2) Bypassing arbitrary scheme for testing purpose only in blink and in V8. 988 // 2) Bypassing arbitrary scheme for testing purpose only in blink and in V8.
983 // TODO(arthursonzogni): url::GetBypassingCSPScheme() is used instead of the 989 // TODO(arthursonzogni): url::GetBypassingCSPScheme() is used instead of the
984 // blink::SchemeRegistry. It contains 1) but not 2). 990 // blink::SchemeRegistry. It contains 1) but not 2).
985 const auto& bypassing_schemes = url::GetCSPBypassingSchemes(); 991 const auto& bypassing_schemes = url::GetCSPBypassingSchemes();
(...skipping 2927 matching lines...) Expand 10 before | Expand all | Expand 10 after
3913 } 3919 }
3914 3920
3915 void RenderFrameHostImpl::ForwardGetInterfaceToRenderFrame( 3921 void RenderFrameHostImpl::ForwardGetInterfaceToRenderFrame(
3916 const std::string& interface_name, 3922 const std::string& interface_name,
3917 mojo::ScopedMessagePipeHandle pipe) { 3923 mojo::ScopedMessagePipeHandle pipe) {
3918 GetRemoteInterfaces()->GetInterface(interface_name, std::move(pipe)); 3924 GetRemoteInterfaces()->GetInterface(interface_name, std::move(pipe));
3919 } 3925 }
3920 #endif 3926 #endif
3921 3927
3922 } // namespace content 3928 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698