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

Unified Diff: content/browser/frame_host/navigation_request.cc

Issue 2628593002: PlzNavigate: check report_raw_headers based on the main frame (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 8a1007d4dd3d6550e05f76faa0f276bcaeedc3b5..67a17b53a8ec579a05981863fe613e76e4466df8 100644
--- a/content/browser/frame_host/navigation_request.cc
+++ b/content/browser/frame_host/navigation_request.cc
@@ -634,7 +634,8 @@ void NavigationRequest::OnStartChecksComplete(
SchemeIs(kGuestScheme);
bool report_raw_headers =
- RenderFrameDevToolsAgentHost::IsNetworkHandlerEnabled(frame_tree_node_);
+ RenderFrameDevToolsAgentHost::IsNetworkHandlerEnabled(
+ frame_tree_node_->frame_tree()->root());
dgozman 2017/01/10 17:36:31 In OOPIF world, this is not necessarily the main f
clamy 2017/01/11 13:47:29 Ok I wasn't sure if I needed to take the top level
loader_ = NavigationURLLoader::Create(
frame_tree_node_->navigator()->GetController()->GetBrowserContext(),
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698