| 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..881744540d239aa3bf3192e1e1e62805b46e5a3d 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());
|
|
|
| @@ -458,7 +458,7 @@ class IPCChannelMojoDeadHandleTest : public IPCChannelMojoTestBase {
|
| const base::ProcessHandle client = client_process().Handle();
|
| // Forces GetFileHandleForProcess() fail. It happens occasionally
|
| // in production, so we should exercise it somehow.
|
| - // TODO(morrita): figure out how to safely test this.
|
| + // TODO(morrita): figure out how to safely test this. See crbug.com/464109.
|
| // ::CloseHandle(client);
|
| host_->OnClientLaunched(client);
|
| return true;
|
| @@ -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 CloseHandle() call in DidStartClient(),
|
| + // which has been disabled since crrev.com/843113003, to
|
| + // make this fail. See crbug.com/464109.
|
| + // EXPECT_FALSE(WaitForClientShutdown());
|
| + WaitForClientShutdown();
|
| EXPECT_TRUE(listener.has_error());
|
|
|
| DestroyChannel();
|
|
|