[Autofill] Migrate ContentAutofillDriver<-->AutofillAgent IPCs to mojo.

content/public/test/mock_render_process_host.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 "content/public/test/mock_render_process_host.h"
 
 #include <algorithm>
 #include <utility>
 #include <vector>
 
@@ skipping 86 matching lines @@
     entry_pair.second->OnMessageReceived(FrameHostMsg_RenderProcessGone(
         entry_pair.first, static_cast<int>(details.status), details.exit_code));
   }
 }
 
 void MockRenderProcessHost::EnableSendQueue() {
 }
 
 bool MockRenderProcessHost::Init() {
   has_connection_ = true;
+  remote_interfaces_.reset(new shell::InterfaceProvider);

[sky, 2016/07/22 16:47:48]

This seems reasonable, but we this class also defines SetRemoteInterfaces. Do we
need both?

[leonhsl(Using Gerrit), 2016/07/23 00:41:45]

Good catch~~ I think we can remove SetRemoteInterfaces now, currently its only
user is EmbeddedWorkerTestHelper, which can instead use this empty
InterfaceProvider we initialized here. And I don't think the initial
InterfaceProvider instance needs to be replaced with new one in normal
scenarios. So let's remove SetRemoteInterfaces. 
WDYT? Thanks.

[leonhsl(Using Gerrit), 2016/07/25 06:58:18]

Sorry I had some misunderstanding about MockRenderProcessHost's usage. "I don't
think the initial InterfaceProvider instance needs to be replaced with new one
in normal scenarios." is not true.  In case of EmbeddedWorkerTestHelper, it does
not call MockRenderProcessHost::Init() at all, so no empty InterfaceProvider
gets created there, EmbeddedWorkerTestHelper just creates its own
InterfaceProvider and calls SetRemoteInterfaces() to set into MRPH.

This means that SetRemoteInterfaces() has its own value and we should keep it.
And what we added here is to provide a default value for |remote_interfaces_|.

   return true;
 }
 
 int MockRenderProcessHost::GetNextRoutingID() {
   return ++prev_routing_id_;
 }
 
 void MockRenderProcessHost::AddRoute(int32_t routing_id,
                                      IPC::Listener* listener) {
   listeners_.AddWithID(listener, routing_id);
@@ skipping 258 matching lines @@
   for (ScopedVector<MockRenderProcessHost>::iterator it = processes_.begin();
        it != processes_.end(); ++it) {
     if (*it == host) {
       processes_.weak_erase(it);
       break;
     }
   }
 }
 
 }  // namespace content