Convert ChromeViewHostMsg_UpdatedCacheStats to use mojo.

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 1314 matching lines @@
 
   AddUIThreadInterface(registry.get(), base::Bind(&FieldTrialRecorder::Create));
 
   AddUIThreadInterface(
       registry.get(),
       base::Bind(
           &memory_instrumentation::CoordinatorImpl::BindCoordinatorRequest,
           base::Unretained(
               memory_instrumentation::CoordinatorImpl::GetInstance())));
 
-  GetContentClient()->browser()->ExposeInterfacesToRenderer(registry.get(),
-                                                            this);
+  associated_registry_.reset(new AssociatedInterfaceRegistryImpl());
+  GetContentClient()->browser()->ExposeInterfacesToRenderer(
+      registry.get(), associated_registry_.get(), this);
 
   ServiceManagerConnection* service_manager_connection =
       BrowserContext::GetServiceManagerConnectionFor(browser_context_);
   std::unique_ptr<ConnectionFilterImpl> connection_filter(
       new ConnectionFilterImpl(child_connection_->child_identity(),
                                std::move(registry)));
   connection_filter_controller_ = connection_filter->controller();
   connection_filter_id_ = service_manager_connection->AddConnectionFilter(
       std::move(connection_filter));
 }
@@ skipping 734 matching lines @@
 
 void RenderProcessHostImpl::OnAssociatedInterfaceRequest(
     const std::string& interface_name,
     mojo::ScopedInterfaceEndpointHandle handle) {
   if (interface_name == mojom::RouteProvider::Name_) {
     if (route_provider_binding_.is_bound())
       return;
     mojom::RouteProviderAssociatedRequest request;
     request.Bind(std::move(handle));
     route_provider_binding_.Bind(std::move(request));
+  } else if (associated_registry_->CanBindRequest(interface_name)) {

[Ken Rockot(use gerrit already), 2017/04/11 22:20:36]

Optional nit: You could add a binder for RouteProvider to associated_registry_
and get rid of the first branch here.

[nigeltao1, 2017/04/20 05:12:36]

Done, by adding some new code further up in this file
(content/browser/renderer_host/render_process_host_impl.cc). There's a little
bit of C++ trickery wrt overloaded method names, so I'm not 100% confident I've
got it right. PTAL.

+    associated_registry_->BindRequest(interface_name, std::move(handle));
   } else {
     LOG(ERROR) << "Request for unknown Channel-associated interface: "
                << interface_name;
   }
 }
 
 void RenderProcessHostImpl::OnChannelConnected(int32_t peer_pid) {
   channel_connected_ = true;
   if (IsReady()) {
     DCHECK(!sent_render_process_ready_);
@@ skipping 970 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