Store response on NavigationRequest earlier.

This allows it to be accessed during GetFrameHostForNavigation, which
is important for making process model decisions based on headers.

BUG=922191

Change-Id: I6e8d2ce0f56ef5329f2b8c84d7d09337524dde89
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592127Reviewed-by: Camille Lamy <clamy@chromium.org>
Commit-Queue: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#656478}

content/browser/frame_host/navigation_request.cc

@@ -1319,6 +1319,8 @@ void NavigationRequest::OnResponseStarted(
   TRACE_EVENT_ASYNC_STEP_INTO0("navigation", "NavigationRequest", this,
                                "OnResponseStarted");
   state_ = RESPONSE_STARTED;
+  response_ = response;
+  ssl_info_ = response->head.ssl_info;
 
   // Check if the response should be sent to a renderer.
   response_should_be_rendered_ =
@@ -1430,11 +1432,8 @@ void NavigationRequest::OnResponseStarted(
           common_params_.previews_state, navigation_handle_.get(),
           response->head.headers.get());
 
-  // Store the response and the URLLoaderClient endpoints until checks have been
-  // processed.
-  response_ = response;
+  // Store the URLLoaderClient endpoints until checks have been processed.
   url_loader_client_endpoints_ = std::move(url_loader_client_endpoints);
-  ssl_info_ = response->head.ssl_info;
 
   subresource_loader_params_ = std::move(subresource_loader_params);