Chromium Code Reviews| 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 cec0f3e61315488bbc3c103f7c1b22af31da6c50..e4cb9222d5b7877bd0a0205b18ea9727eba6c83b 100644 |
| --- a/mojo/edk/system/waiter_test_utils.h |
| +++ b/mojo/edk/system/waiter_test_utils.h |
| @@ -29,19 +29,19 @@ namespace test { |
| // |
| // MojoResult result; |
| // { |
| -// WaiterList waiter_list; |
| +// AwakableList awakable_list; |
| // test::SimpleWaiterThread thread(&result); |
| -// waiter_list.AddWaiter(thread.waiter(), ...); |
| +// awakable_list.Add(thread.waiter(), ...); |
| // thread.Start(); |
| // ... some stuff to wake the waiter ... |
| -// waiter_list.RemoveWaiter(thread.waiter()); |
| +// awakable_list.Remove(thread.waiter()); |
| // } // Join |thread|. |
| // EXPECT_EQ(..., result); |
| // |
| // There's a bit of unrealism in its use: In this sort of usage, calls such as |
| -// |Waiter::Init()|, |AddWaiter()|, and |RemoveWaiter()| are done in the main |
| -// (test) thread, not the waiter thread (as would actually happen in real code). |
| -// (We accept this unrealism for simplicity, since |WaiterList| is |
| +// |Waiter::Init()|, |AddAwakable()|, and |RemoveAwakable()| are done in the |
| +// main (test) thread, not the waiter thread (as would actually happen in real |
| +// code). (We accept this unrealism for simplicity, since |AwakableList| is |
|
viettrungluu
2014/12/03 23:07:47
nit: one space after period, please :)
|
| // thread-unsafe so making it more realistic would require adding nontrivial |
| // synchronization machinery.) |
| class SimpleWaiterThread : public base::SimpleThread { |