Plumb NQP to context and to http_proxy_client_socket_pool

chrome/browser/profiles/profile_io_data.cc

Index: chrome/browser/profiles/profile_io_data.cc
diff --git a/chrome/browser/profiles/profile_io_data.cc b/chrome/browser/profiles/profile_io_data.cc
index 644fd76b0ff6405def02008a4b389b14819acde0..8e176cf92790a4ac5081f026b0c8ae3aa7ad3452 100644
--- a/chrome/browser/profiles/profile_io_data.cc
+++ b/chrome/browser/profiles/profile_io_data.cc
@@ -1312,11 +1312,13 @@ ProfileIOData::CreateHttpNetworkSession(
   net::HttpNetworkSession::Context session_context;
   net::URLRequestContextBuilder::SetHttpNetworkSessionComponents(
       context, &session_context);
-  if (!IsOffTheRecord() && io_thread->globals()->network_quality_estimator) {
-    session_context.socket_performance_watcher_factory =
-        io_thread->globals()
-            ->network_quality_estimator->GetSocketPerformanceWatcherFactory();
-  }
+  DCHECK(io_thread->globals()->network_quality_estimator);
+  session_context.socket_performance_watcher_factory =
+      io_thread->globals()
+          ->network_quality_estimator->GetSocketPerformanceWatcherFactory();

[mmenke, 2017/06/05 20:04:25]

Is hooking this up in incognito mode a critical part of this CL?  Whether it is
or not, mind landing this change separately, and making sure you have a signoff
from the privacy team?  I don't see an issue, but I'll defer to them.

[tbansal1, 2017/06/07 18:13:44]

Done.

+  session_context.network_quality_estimator =
+      io_thread->globals()->network_quality_estimator.get();

[mmenke, 2017/06/05 20:04:25]

Not needed, since it's plugged into SetHttpNetworkSessionComponents, a couple
lines up.

[tbansal1, 2017/06/07 18:13:44]

Done.

+
   if (data_reduction_proxy_io_data_.get()) {
     session_context.proxy_delegate =
         data_reduction_proxy_io_data_->proxy_delegate();