[GRC] Hook up the process-level CoordinationUnit

content/browser/renderer_host/render_process_host_impl.cc

 // Copyright 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.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 
 #include <algorithm>
@@ skipping 835 matching lines @@
   using ProcessID = int;
   using SiteProcessIDPair = std::pair<GURL, ProcessID>;
   using SiteProcessIDPairSet = std::set<SiteProcessIDPair>;
 
   // Use std::set because duplicates don't need to be tracked separately (eg.,
   // service workers for the same site in the same process). It is sorted in the
   // order of insertion.
   SiteProcessIDPairSet site_process_set_;
 };
 
+void CreateResourceCoordinatorProcessInterface(

[nasko, 2017/06/29 17:24:28]

nit: I'd move this to be closer to CreateMemoryCoordinatorHandle, as they are
similar type of functionality - coordination related.

[lpy, 2017/06/29 22:00:35]

Done.

+    RenderProcessHostImpl* render_process_host,
+    const service_manager::BindSourceInfo& source_info,
+    resource_coordinator::mojom::CoordinationUnitRequest request) {
+  render_process_host->GetProcessResourceCoordinator()->service()->AddBinding(
+      std::move(request));
+}
+
 }  // namespace
 
 RendererMainThreadFactoryFunction g_renderer_main_thread_factory = NULL;
 
 base::MessageLoop* g_in_process_thread;
 
 // Stores the maximum number of renderer processes the content module can
 // create.
 static size_t g_max_renderer_count_override = 0;
 
@@ skipping 789 matching lines @@
   registry->AddInterface(base::Bind(&RenderProcessHostImpl::CreateMusGpuRequest,
                                     base::Unretained(this)));
 
   registry->AddInterface(
       base::Bind(&VideoCaptureHost::Create,
                  BrowserMainLoop::GetInstance()->media_stream_manager()));
 
   registry->AddInterface(
       base::Bind(&metrics::CreateSingleSampleMetricsProvider));
 
+  registry->AddInterface(base::Bind(&CreateResourceCoordinatorProcessInterface,
+                                    base::Unretained(this)));

[nasko, 2017/06/29 21:52:46]

Let's make this conditional on the GRC feature being enabled.

[lpy, 2017/06/29 22:00:35]

Done.

+
   if (base::FeatureList::IsEnabled(features::kOffMainThreadFetch)) {
     scoped_refptr<ServiceWorkerContextWrapper> service_worker_context(
         static_cast<ServiceWorkerContextWrapper*>(
             storage_partition_impl_->GetServiceWorkerContext()));
     registry->AddInterface(
         base::Bind(&WorkerURLLoaderFactoryProviderImpl::Create, GetID(),
                    resource_message_filter_, service_worker_context));
   }
 
   // This is to support usage of WebSockets in cases in which there is no
@@ skipping 2112 matching lines @@
   LOG(ERROR) << "Terminating render process for bad Mojo message: " << error;
 
   // The ReceivedBadMessage call below will trigger a DumpWithoutCrashing.
   // Capture the error message in a crash key value.
   base::debug::ScopedCrashKey error_key_value("mojo-message-error", error);
   bad_message::ReceivedBadMessage(render_process_id,
                                   bad_message::RPH_MOJO_PROCESS_ERROR);
 }
 
 }  // namespace content