Merge CrossSiteResourceHandler and NavigationResourceThrottle

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 "content/browser/devtools/render_frame_devtools_agent_host.h"
 #include "content/browser/frame_host/frame_tree.h"
@@ skipping 318 matching lines @@
   }
 
   if (navigation_data)
     navigation_handle_->set_navigation_data(std::move(navigation_data));
 
   // Store the response and the StreamHandle until checks have been processed.
   response_ = response;
   body_ = std::move(body);
 
   // Check if the navigation should be allowed to proceed.
+  // TODO(clamy): pass the right values for request_id, is_download and
+  // is_stream.

[nasko, 2016/09/08 23:45:39]

Do you plan to resolve this TODO before committing this CL or do it in a
separate one?

[clamy, 2016/09/09 15:06:41]

No I plan to do it in a separate one. Since this is for transfers, I don't need
to pass them in PlzNavigate. However we should do it in the future for the XFO
throttle to work.

   navigation_handle_->WillProcessResponse(
       render_frame_host, response->head.headers.get(), ssl_status,
+      GlobalRequestID(), common_params_.should_replace_current_entry, false,
+      false, base::Closure(),
       base::Bind(&NavigationRequest::OnWillProcessResponseChecksComplete,
                  base::Unretained(this)));
 }
 
 void NavigationRequest::OnRequestFailed(bool has_stale_copy_in_cache,
                                         int net_error) {
   DCHECK(state_ == STARTED);
   state_ = FAILED;
   navigation_handle_->set_net_error_code(static_cast<net::Error>(net_error));
   frame_tree_node_->navigator()->FailedNavigation(
@@ skipping 109 matching lines @@
 
   TransferNavigationHandleOwnership(render_frame_host);
   render_frame_host->CommitNavigation(response_.get(), std::move(body_),
                                       common_params_, request_params_,
                                       is_view_source_);
 
   frame_tree_node_->ResetNavigationRequest(true);
 }
 
 }  // namespace content