Plumb NQP to context and to http_proxy_client_socket_pool

chrome/browser/profiles/profile_io_data.cc

 // Copyright (c) 2012 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 "chrome/browser/profiles/profile_io_data.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 1294 matching lines @@
 std::unique_ptr<net::HttpNetworkSession>
 ProfileIOData::CreateHttpNetworkSession(
     const ProfileParams& profile_params) const {
   net::URLRequestContext* context = main_request_context();
 
   IOThread* const io_thread = profile_params.io_thread;
 
   net::HttpNetworkSession::Context session_context;
   net::URLRequestContextBuilder::SetHttpNetworkSessionComponents(
       context, &session_context);
-  if (!IsOffTheRecord() && io_thread->globals()->network_quality_estimator) {
+  DCHECK(io_thread->globals()->network_quality_estimator);

[mmenke, 2017/06/07 21:31:25]

Does this DCHECK get us anything?

[tbansal1, 2017/06/08 05:01:49]

Done.

+  if (!IsOffTheRecord()) {
     session_context.socket_performance_watcher_factory =
         io_thread->globals()
             ->network_quality_estimator->GetSocketPerformanceWatcherFactory();
   }
   if (data_reduction_proxy_io_data_.get()) {
     session_context.proxy_delegate =
         data_reduction_proxy_io_data_->proxy_delegate();
   }
 
   return std::unique_ptr<net::HttpNetworkSession>(new net::HttpNetworkSession(
@@ skipping 30 matching lines @@
 void ProfileIOData::SetCookieSettingsForTesting(
     content_settings::CookieSettings* cookie_settings) {
   DCHECK(!cookie_settings_.get());
   cookie_settings_ = cookie_settings;
 }
 
 policy::URLBlacklist::URLBlacklistState ProfileIOData::GetURLBlacklistState(
     const GURL& url) const {
   return url_blacklist_manager_->GetURLBlacklistState(url);
 }