Fix assumption in mojo binding class Connector that message pipes notice the other end's closing synchronously.
This was causing mojo_public_bindings_unittests' MultiplexRouterTest.BasicRequestResponse and MultiplexRouterTest.RequestWithNoReceiver to fail with the new EDK. The problem is that in Connector::HandleError message_pipe_ is reset to a dummy message pipe whose other end is closed immediately. In the old EDK, the live end notices that the peer is closed synchronously. In the new EDK this happens asynchronously because of thread hops to the IO thread. The fix is to simply ensure we don't watch the message pipe handle twice (since it's not cancelled immediately).
BUG=561803
Review URL: https://codereview.chromium.org/1557753002
Cr-Commit-Position: refs/heads/master@{#367327}
CrOS-Libchrome-Original-Commit: b0c8bd36ad840b609050caee081fa73e0e212559
1 file changed