Make sure channel-associated interface pointers are always safe to call.

ipc/ipc_channel_mojo.cc

Index: ipc/ipc_channel_mojo.cc
diff --git a/ipc/ipc_channel_mojo.cc b/ipc/ipc_channel_mojo.cc
index d0129e3fe0833596ac3b576886f6d3e4ca54fd8f..402715f93edb226693019651402582ea231b1114 100644
--- a/ipc/ipc_channel_mojo.cc
+++ b/ipc/ipc_channel_mojo.cc
@@ -479,8 +479,11 @@ void ChannelMojo::AddGenericAssociatedInterface(
 void ChannelMojo::GetGenericRemoteAssociatedInterface(
     const std::string& name,
     mojo::ScopedInterfaceEndpointHandle handle) {
-  if (message_reader_)
+  if (message_reader_) {
     message_reader_->GetRemoteInterface(name, std::move(handle));
+  } else {
+    mojo::GetIsolatedInterface(std::move(handle));

[Ken Rockot(use gerrit already), 2017/03/24 19:20:03]

nit: I might consider documenting here why we do this and why it's OK.

[yzshen1, 2017/03/25 07:18:15]

Done.

+  }
 }
 
 }  // namespace IPC