Fix a couple of ipc_channel_mojo_unittest failures.

ipc/mojo/ipc_channel_mojo_unittest.cc

 // Copyright 2014 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 "ipc/mojo/ipc_channel_mojo.h"
 
 #include "base/base_paths.h"
 #include "base/files/file.h"
 #include "base/message_loop/message_loop.h"
 #include "base/path_service.h"
@@ skipping 413 matching lines @@
 
   base::MessageLoop::current()->Run();
   this->channel()->Close();
 
   EXPECT_TRUE(WaitForClientShutdown());
   DestroyChannel();
 }
 
 MULTIPROCESS_IPC_TEST_CLIENT_MAIN(IPCChannelMojoTestSendMessagePipeClient) {
   ListenerThatExpectsMessagePipe listener;
-  ChannelClient client(&listener, "IPCChannelMojoTestSendPlatformHandleClient");
+  ChannelClient client(&listener, "IPCChannelMojoTestSendMessagePipeClient");
   client.Connect();
   listener.set_sender(client.channel());
 
   base::MessageLoop::current()->Run();
 
   client.Close();
 
   return 0;
 }
 
@@ skipping 30 matching lines @@
   // Set up IPC channel and start client.
   ListenerExpectingErrors listener;
   CreateChannel(&listener);
   ASSERT_TRUE(ConnectChannel());
 
   ASSERT_TRUE(StartClient());
   base::MessageLoop::current()->Run();
 
   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();
 }
 
 MULTIPROCESS_IPC_TEST_CLIENT_MAIN(IPCChannelMojoTestDoNothingClient) {
   ListenerThatQuits listener;
   ChannelClient client(&listener, "IPCChannelMojoTestDoNothingClient");
   client.Connect();
 
@@ skipping 168 matching lines @@
   base::MessageLoop::current()->Run();
 
   client.Close();
 
   return 0;
 }
 
 #endif // OS_LINUX
 
 }  // namespace