Associated interfaces sent over closed associated interfaces do not trigger connection errors |
||
Issue descriptionWhen a message is received for an associated interface without a client, the message is dropped. If the message contains an associated interface, the other end of the interface is not notified with a connection error. See https://codereview.chromium.org/2665053002 for tests that trigger this.
,
Jan 31 2017
It will be fixed by this CL: https://codereview.chromium.org/2660733002/
,
Feb 2 2017
My fix has landed and the tests now pass. Would you please land your test CL, Sam? Thanks for writing the tests!
,
Feb 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5c8a6c605dc41954b355cd30eb926b8c2143a148 commit 5c8a6c605dc41954b355cd30eb926b8c2143a148 Author: sammc <sammc@chromium.org> Date: Sat Feb 04 01:33:38 2017 Add tests for detecting lost associated interfaces. BUG= 686955 Review-Url: https://codereview.chromium.org/2665053002 Cr-Commit-Position: refs/heads/master@{#448139} [modify] https://crrev.com/5c8a6c605dc41954b355cd30eb926b8c2143a148/ipc/ipc_channel_mojo_unittest.cc [modify] https://crrev.com/5c8a6c605dc41954b355cd30eb926b8c2143a148/ipc/ipc_test.mojom [modify] https://crrev.com/5c8a6c605dc41954b355cd30eb926b8c2143a148/mojo/public/cpp/bindings/tests/associated_interface_unittest.cc [modify] https://crrev.com/5c8a6c605dc41954b355cd30eb926b8c2143a148/mojo/public/interfaces/bindings/tests/test_associated_interfaces.mojom
,
Feb 4 2017
|
||
►
Sign in to add a comment |
||
Comment 1 Deleted