DescriptionFix 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
Committed: https://crrev.com/b0c8bd36ad840b609050caee081fa73e0e212559
Cr-Commit-Position: refs/heads/master@{#367327}
Patch Set 1 #
Messages
Total messages: 11 (6 generated)
|