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

Unified Diff: mojo/edk/system/waiter_test_utils.h

Issue 1361143004: EDK: Add a mojo::test::SimpleTestThread (and use it). (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 3 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/waiter_test_utils.h
diff --git a/mojo/edk/system/waiter_test_utils.h b/mojo/edk/system/waiter_test_utils.h
index 350d87f286ab8ce692af76f62b2a00484767187b..ee2f68c721f9b823d198c8f6108ad625f77caeb1 100644
--- a/mojo/edk/system/waiter_test_utils.h
+++ b/mojo/edk/system/waiter_test_utils.h
@@ -8,10 +8,10 @@
#include <stdint.h>
#include "base/memory/ref_counted.h"
-#include "base/threading/simple_thread.h"
#include "mojo/edk/system/dispatcher.h"
#include "mojo/edk/system/handle_signals_state.h"
#include "mojo/edk/system/waiter.h"
+#include "mojo/edk/test/simple_test_thread.h"
#include "mojo/public/c/system/types.h"
#include "mojo/public/cpp/system/macros.h"
@@ -43,7 +43,7 @@ namespace test {
// code). (We accept this unrealism for simplicity, since |AwakableList| is
// thread-unsafe so making it more realistic would require adding nontrivial
// synchronization machinery.)
-class SimpleWaiterThread : public base::SimpleThread {
+class SimpleWaiterThread : public mojo::test::SimpleTestThread {
public:
// For the duration of the lifetime of this object, |*result| belongs to it
// (in the sense that it will write to it whenever it wants).
@@ -65,7 +65,7 @@ class SimpleWaiterThread : public base::SimpleThread {
// This is a more complex and realistic thread that has a |Waiter|, on which it
// waits for the given deadline (with the given flags). Unlike
// |SimpleWaiterThread|, it requires the machinery of |Dispatcher|.
-class WaiterThread : public base::SimpleThread {
+class WaiterThread : public mojo::test::SimpleTestThread {
public:
// Note: |*did_wait_out|, |*result_out|, |*context_out| and
// |*signals_state_out| "belong" to this object (i.e., may be modified by, on

Powered by Google App Engine
This is Rietveld 408576698