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

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

Issue 2750373002: Revert of Mojo: Armed Watchers (Closed)
Patch Set: Created 3 years, 9 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
« no previous file with comments | « mojo/edk/system/data_pipe_producer_dispatcher.cc ('k') | mojo/edk/system/dispatcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/system/data_pipe_unittest.cc
diff --git a/mojo/edk/system/data_pipe_unittest.cc b/mojo/edk/system/data_pipe_unittest.cc
index 114749628a363ce2a11dbf4662aeaed443ccacd7..610aeac67df924b0896dc4761301a075c5f727f6 100644
--- a/mojo/edk/system/data_pipe_unittest.cc
+++ b/mojo/edk/system/data_pipe_unittest.cc
@@ -21,7 +21,7 @@
#include "mojo/public/c/system/data_pipe.h"
#include "mojo/public/c/system/functions.h"
#include "mojo/public/c/system/message_pipe.h"
-#include "mojo/public/cpp/system/simple_watcher.h"
+#include "mojo/public/cpp/system/watcher.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace mojo {
@@ -2029,7 +2029,7 @@
base::MessageLoop message_loop;
- // Wait on producer 1 and consumer 1 using SimpleWatchers.
+ // Wait on producer 1 and consumer 1 using Watchers.
{
base::RunLoop run_loop;
int count = 0;
@@ -2040,16 +2040,12 @@
loop->Quit();
},
&run_loop, &count);
- SimpleWatcher producer_watcher(FROM_HERE,
- SimpleWatcher::ArmingPolicy::AUTOMATIC);
- SimpleWatcher consumer_watcher(FROM_HERE,
- SimpleWatcher::ArmingPolicy::AUTOMATIC);
- producer_watcher.Watch(Handle(producers[1]), MOJO_HANDLE_SIGNAL_PEER_CLOSED,
- callback);
- consumer_watcher.Watch(Handle(consumers[1]), MOJO_HANDLE_SIGNAL_PEER_CLOSED,
- callback);
+ Watcher producer_watcher(FROM_HERE), consumer_watcher(FROM_HERE);
+ producer_watcher.Start(
+ Handle(producers[1]), MOJO_HANDLE_SIGNAL_PEER_CLOSED, callback);
+ consumer_watcher.Start(
+ Handle(consumers[1]), MOJO_HANDLE_SIGNAL_PEER_CLOSED, callback);
run_loop.Run();
- EXPECT_EQ(2, count);
}
// Wait on producer 2 by polling with MojoWriteData.
« no previous file with comments | « mojo/edk/system/data_pipe_producer_dispatcher.cc ('k') | mojo/edk/system/dispatcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698