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

services/shell/public/cpp/lib/interface_provider.cc

Index: services/shell/public/cpp/lib/interface_provider.cc
diff --git a/services/shell/public/cpp/lib/interface_provider.cc b/services/shell/public/cpp/lib/interface_provider.cc
index 209d356ac476bb6a79d8cfc51f389500c8b297b7..63a8753e5a3127348fcf7f70c1b66dfaeb6c4ab3 100644
--- a/services/shell/public/cpp/lib/interface_provider.cc
+++ b/services/shell/public/cpp/lib/interface_provider.cc
@@ -19,7 +19,8 @@ void InterfaceProvider::Bind(mojom::InterfaceProviderPtr interface_provider) {
 
 void InterfaceProvider::SetConnectionLostClosure(
     const base::Closure& connection_lost_closure) {
-  interface_provider_.set_connection_error_handler(connection_lost_closure);
+  if (interface_provider_)

[yzshen1, 2016/06/29 22:25:44]

Why this check is needed? It seems this is never null.

[leonhsl(Using Gerrit), 2016/06/29 23:28:35]

With the render_frame_host_impl.cc changes, InterfaceProvider could be
constructed but with no binding, so added the check here. 

[yzshen1, 2016/06/30 16:58:17]

But the GetProxy() at line 10 guarantees that it is not null. (The other end is
not bound, but that is okay, method calls are cached.)

Have I missed anything?

[leonhsl(Using Gerrit), 2016/07/01 13:04:59]

Yeah you are right.. As pending_request_ is used now, this check has become
unnecessary. Thanks! Done.

+    interface_provider_.set_connection_error_handler(connection_lost_closure);
 }
 
 base::WeakPtr<InterfaceProvider> InterfaceProvider::GetWeakPtr() {
@@ -35,7 +36,8 @@ void InterfaceProvider::GetInterface(
     it->second.Run(std::move(request_handle));
     return;
   }
-  interface_provider_->GetInterface(name, std::move(request_handle));
+  if (interface_provider_)

[yzshen1, 2016/06/29 22:25:44]

ditto.

[leonhsl(Using Gerrit), 2016/07/01 13:04:59]

Done.

+    interface_provider_->GetInterface(name, std::move(request_handle));
 }
 
 void InterfaceProvider::ClearBinders() {