Hook up ProfileIOData's URLRequestContext to a NetworkService.

chrome/browser/chrome_content_browser_client.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/chrome_content_browser_client.h"
 
 #include <map>
 #include <set>
 #include <utility>
 #include <vector>
@@ skipping 35 matching lines @@
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/download/download_prefs.h"
 #include "chrome/browser/font_family_cache.h"
 #include "chrome/browser/media/router/media_router_feature.h"
 #include "chrome/browser/media/router/presentation_service_delegate_impl.h"
 #include "chrome/browser/media/router/receiver_presentation_service_delegate_impl.h"
 #include "chrome/browser/media/webrtc/media_capture_devices_dispatcher.h"
 #include "chrome/browser/memory/chrome_memory_coordinator_delegate.h"
 #include "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
 #include "chrome/browser/nacl_host/nacl_browser_delegate_impl.h"
+#include "chrome/browser/net/profile_network_context_service.h"
+#include "chrome/browser/net/profile_network_context_service_factory.h"
 #include "chrome/browser/net_benchmarking.h"
 #include "chrome/browser/notifications/platform_notification_service_impl.h"
 #include "chrome/browser/page_load_metrics/experiments/delay_navigation_throttle.h"
 #include "chrome/browser/page_load_metrics/metrics_navigation_throttle.h"
 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 #include "chrome/browser/permissions/permission_context_base.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/prerender/prerender_final_status.h"
 #include "chrome/browser/prerender/prerender_manager.h"
@@ skipping 2613 matching lines @@
 void ChromeContentBrowserClient::GetURLRequestAutoMountHandlers(
     std::vector<storage::URLRequestAutoMountHandler>* handlers) {
   for (size_t i = 0; i < extra_parts_.size(); ++i)
     extra_parts_[i]->GetURLRequestAutoMountHandlers(handlers);
 }
 
 ::rappor::RapporService* ChromeContentBrowserClient::GetRapporService() {
   return g_browser_process->rappor_service();
 }
 
+content::mojom::NetworkContextPtr
+ChromeContentBrowserClient::CreateNetworkContext(
+    content::BrowserContext* context,
+    bool in_memory,
+    const base::FilePath& relative_partition_path) {
+  if (relative_partition_path.empty()) {
+    return ProfileNetworkContextServiceFactory::GetForContext(context)
+        ->CreateMainNetworkContext();
+  }
+  // TODO(mmenke):  Implement this.

[Randy Smith (Not in Mondays), 2017/07/20 17:38:34]

nit, suggestion: Enhance TODO to indicate what's missing from this path.

[Randy Smith (Not in Mondays), 2017/07/20 17:38:34]

Confirming that this TODO is basically parallel to the TODO that used to be in
StoragePartitionImpl::Create() around filling in context_param?

[mmenke, 2017/07/20 18:31:30]

Not quite analogous - I'm not sure it will even make sense to populate them in
content/ (As opposed to having individual embedders populate them), though not
at all certain of that.

[mmenke, 2017/07/20 18:31:30]

Done.  Left this pretty general ("Implement this once
ProfileNetworkContextServiceFactory can create a fully functional NetworkContext
for Apps when the network service is disabled.").  We could implement it before
then, but I'm not sure how much benefit there would be.

+  return ContentBrowserClient::CreateNetworkContext(context, in_memory,
+                                                    relative_partition_path);
+}
+
 void ChromeContentBrowserClient::GetAdditionalFileSystemBackends(
     content::BrowserContext* browser_context,
     const base::FilePath& storage_partition_path,
     std::vector<std::unique_ptr<storage::FileSystemBackend>>*
         additional_backends) {
 #if defined(OS_CHROMEOS)
   storage::ExternalMountPoints* external_mount_points =
       content::BrowserContext::GetMountPoints(browser_context);
   DCHECK(external_mount_points);
   auto backend = base::MakeUnique<chromeos::FileSystemBackend>(
@@ skipping 750 matching lines @@
   // |safe_browsing_service_| may be unavailable in tests.
   if (safe_browsing_service_ && !safe_browsing_url_checker_delegate_) {
     safe_browsing_url_checker_delegate_ =
         new safe_browsing::UrlCheckerDelegateImpl(
             safe_browsing_service_->database_manager(),
             safe_browsing_service_->ui_manager());
   }
 
   return safe_browsing_url_checker_delegate_.get();
 }