Fix a couple of ipc_channel_mojo_unittest failures.

ipc/mojo/ipc_channel_mojo_unittest.cc

Index: ipc/mojo/ipc_channel_mojo_unittest.cc
diff --git a/ipc/mojo/ipc_channel_mojo_unittest.cc b/ipc/mojo/ipc_channel_mojo_unittest.cc
index c6d1894ec23180c0b83a14663407662fd2eaa67f..c67ef38b84b6643bc122044a3cf1d8c4875d2b96 100644
--- a/ipc/mojo/ipc_channel_mojo_unittest.cc
+++ b/ipc/mojo/ipc_channel_mojo_unittest.cc
@@ -431,7 +431,7 @@ TEST_F(IPCChannelMojoTest, SendMessagePipe) {
 
 MULTIPROCESS_IPC_TEST_CLIENT_MAIN(IPCChannelMojoTestSendMessagePipeClient) {
   ListenerThatExpectsMessagePipe listener;
-  ChannelClient client(&listener, "IPCChannelMojoTestSendPlatformHandleClient");
+  ChannelClient client(&listener, "IPCChannelMojoTestSendMessagePipeClient");
   client.Connect();
   listener.set_sender(client.channel());
 
@@ -482,9 +482,11 @@ TEST_F(IPCChannelMojoDeadHandleTest, InvalidClientHandle) {
 
   this->channel()->Close();
 
-  // WaitForClientShutdown() fails as client_hanadle() is already
-  // closed.
-  EXPECT_FALSE(WaitForClientShutdown());
+  // TODO(morrita): We need loseHandle() call on DidStartClient(),

[viettrungluu, 2015/03/04 22:12:41]

I think you mean "CloseHandle"?

(Also "on" -> "in"?)

[Hajime Morrita, 2015/03/04 23:09:07]

Done.

+  // which has been disabled since crrev.com/843113003, to

[viettrungluu, 2015/03/04 22:12:41]

Sorry, what has been disabled exactly?

(Also, "to make this fail" -> "making this fail"?)

[Hajime Morrita, 2015/03/04 23:09:07]

The series of CLs is apparently attempting to wrap the handle
with a scoped object. So closing it will make the cleanup fail.

+  // make this fail.
+  // EXPECT_FALSE(WaitForClientShutdown());

[viettrungluu, 2015/03/04 22:12:41]

Could you file a bug and put the bug number/URL in the TODO?

[Hajime Morrita, 2015/03/04 23:09:07]

Done.

+  WaitForClientShutdown();
   EXPECT_TRUE(listener.has_error());
 
   DestroyChannel();