Fix a couple of ipc_channel_mojo_unittest failures.

Fix a couple of ipc_channel_mojo_unittest failures.

This should've been covered by commit bot.
The bot configuration chagne will come with a separate CL.

R=viettrungluu@chromium.org, agl@chromium.org
TEST=ipc_channel_mojo_unittest.cc
BUG=

Committed: https://crrev.com/e73c0b43a0fb4c8a93ba8e8155111a285294117f
Cr-Commit-Position: refs/heads/master@{#319199}

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

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
File ipc/mojo/ipc_channel_mojo_unittest.cc (right):

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
ipc/mojo/ipc_channel_mojo_unittest.cc:485: // TODO(morrita): We need
loseHandle() call on DidStartClient(),
I think you mean "CloseHandle"?

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

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
ipc/mojo/ipc_channel_mojo_unittest.cc:486: // which has been disabled since
crrev.com/843113003, to
Sorry, what has been disabled exactly?

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

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
ipc/mojo/ipc_channel_mojo_unittest.cc:488: //
EXPECT_FALSE(WaitForClientShutdown());
Could you file a bug and put the bug number/URL in the TODO?

[Hajime Morrita, 2015-03-04 23:09:08]

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
File ipc/mojo/ipc_channel_mojo_unittest.cc (right):

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
ipc/mojo/ipc_channel_mojo_unittest.cc:485: // TODO(morrita): We need
loseHandle() call on DidStartClient(),
On 2015/03/04 22:12:41, viettrungluu wrote:
> I think you mean "CloseHandle"?
> 
> (Also "on" -> "in"?)

Done.

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
ipc/mojo/ipc_channel_mojo_unittest.cc:486: // which has been disabled since
crrev.com/843113003, to
On 2015/03/04 22:12:41, viettrungluu wrote:
> Sorry, what has been disabled exactly?
> 
> (Also, "to make this fail" -> "making this fail"?)
The series of CLs is apparently attempting to wrap the handle
with a scoped object. So closing it will make the cleanup fail.

https://codereview.chromium.org/976043003/diff/1/ipc/mojo/ipc_channel_mojo_un...
ipc/mojo/ipc_channel_mojo_unittest.cc:488: //
EXPECT_FALSE(WaitForClientShutdown());
On 2015/03/04 22:12:41, viettrungluu wrote:
> Could you file a bug and put the bug number/URL in the TODO?

Done.

[viettrungluu, 2015-03-05 01:11:35]

lgtm

[Hajime Morrita, 2015-03-05 01:34:45]

The CQ bit was checked by morrita@chromium.org

[commit-bot: I haz the power, 2015-03-05 01:35:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/976043003/20001

[commit-bot: I haz the power, 2015-03-05 02:55:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-05 02:56:27]

Patchset 2 (id:??) landed as
https://crrev.com/e73c0b43a0fb4c8a93ba8e8155111a285294117f
Cr-Commit-Position: refs/heads/master@{#319199}