Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(975)

Unified Diff: mojo/edk/system/ipc_support_unittest.cc

Issue 2075353002: Plumb the handle signals state out of Waiter::Wait(). (Closed) Base URL: https://github.com/domokit/mojo.git@work795_wait_set_4.3-x-work794_wait_set_4.2
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/edk/system/ipc_support_unittest.cc
diff --git a/mojo/edk/system/ipc_support_unittest.cc b/mojo/edk/system/ipc_support_unittest.cc
index 1c0375085077629f6d488a6c7cd7ca9c78e9d0d4..f271c715db2f9ffa2feb3850bf2983574eb773d1 100644
--- a/mojo/edk/system/ipc_support_unittest.cc
+++ b/mojo/edk/system/ipc_support_unittest.cc
@@ -62,7 +62,8 @@ void TestWriteReadMessage(MessagePipeDispatcher* write_mp,
MOJO_WRITE_MESSAGE_FLAG_NONE));
// Wait for it to arrive.
- EXPECT_EQ(MOJO_RESULT_OK, waiter.Wait(test::ActionTimeout(), nullptr));
+ EXPECT_EQ(MOJO_RESULT_OK,
+ waiter.Wait(test::ActionTimeout(), nullptr, nullptr));
read_mp->RemoveAwakable(&waiter, nullptr);
// Read the message from the read end.
@@ -109,7 +110,8 @@ RefPtr<MessagePipeDispatcher> SendMessagePipeDispatcher(
mp_handle_to_send.reset();
// Wait for it to arrive.
- CHECK_EQ(waiter.Wait(test::ActionTimeout(), nullptr), MOJO_RESULT_OK);
+ CHECK_EQ(waiter.Wait(test::ActionTimeout(), nullptr, nullptr),
+ MOJO_RESULT_OK);
read_mp->RemoveAwakable(&waiter, nullptr);
// Read the message from the read end.

Powered by Google App Engine
This is Rietveld 408576698