PlzNavigate: check report_raw_headers based on the main frame

content/browser/frame_host/navigation_request.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/frame_host/navigation_request.h"
 
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "content/browser/appcache/appcache_navigation_handle.h"
@@ skipping 616 matching lines @@
 
   std::unique_ptr<NavigationUIData> navigation_ui_data;
   if (navigation_handle_->navigation_ui_data())
     navigation_ui_data = navigation_handle_->navigation_ui_data()->Clone();
 
   bool is_for_guests_only =
       navigation_handle_->GetStartingSiteInstance()->GetSiteURL().
           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 frame. We should instead alter
RFDTAH::IsNetworkHandlerEnabled to climb up the hierarchy. Something like this:
https://cs.chromium.org/chromium/src/content/browser/devtools/render_frame_de...

[clamy, 2017/01/11 13:47:29]

Ok I wasn't sure if I needed to take the top level frame or do a walk of the
frame tree. I've opted to add this to RenderFrameDevToolsAgentHost instead,
since this seems to be a detail of the DevTools implementation.

 
   loader_ = NavigationURLLoader::Create(
       frame_tree_node_->navigator()->GetController()->GetBrowserContext(),
       base::MakeUnique<NavigationRequestInfo>(
           common_params_, begin_params_, first_party_for_cookies,
           frame_tree_node_->IsMainFrame(), parent_is_main_frame,
           IsSecureFrame(frame_tree_node_->parent()),
           frame_tree_node_->frame_tree_node_id(), is_for_guests_only,
           report_raw_headers, navigating_frame_host->GetVisibilityState()),
       std::move(navigation_ui_data),
@@ skipping 66 matching lines @@
   DCHECK_EQ(request_params_.has_user_gesture, begin_params_.has_user_gesture);
 
   render_frame_host->CommitNavigation(response_.get(), std::move(body_),
                                       common_params_, request_params_,
                                       is_view_source_);
 
   frame_tree_node_->ResetNavigationRequest(true);
 }
 
 }  // namespace content