| Index: mojo/edk/system/handle_table_unittest.cc
|
| diff --git a/mojo/edk/system/handle_table_unittest.cc b/mojo/edk/system/handle_table_unittest.cc
|
| index 33c983d07b21662dbe52cd068faa960d4125d88a..55af970f05dbb4133c710672d4cb6c47893cf1fe 100644
|
| --- a/mojo/edk/system/handle_table_unittest.cc
|
| +++ b/mojo/edk/system/handle_table_unittest.cc
|
| @@ -23,8 +23,8 @@ TEST(HandleTableTest, Basic) {
|
|
|
| RefPtr<Dispatcher> d = MakeRefCounted<test::MockSimpleDispatcher>();
|
|
|
| - MojoHandle h = ht.AddHandle(Handle(d.Clone(), MOJO_HANDLE_RIGHT_NONE));
|
| - ASSERT_NE(h, MOJO_HANDLE_INVALID);
|
| + MojoHandle hv = ht.AddHandle(Handle(d.Clone(), MOJO_HANDLE_RIGHT_NONE));
|
| + ASSERT_NE(hv, MOJO_HANDLE_INVALID);
|
|
|
| // Save the pointer value (without taking a ref), so we can check that we get
|
| // the same object back.
|
| @@ -32,18 +32,19 @@ TEST(HandleTableTest, Basic) {
|
| // Reset this, to make sure that the handle table takes a ref.
|
| d = nullptr;
|
|
|
| - EXPECT_EQ(MOJO_RESULT_OK, ht.GetDispatcher(h, &d));
|
| - EXPECT_EQ(d.get(), dv);
|
| + Handle h;
|
| + EXPECT_EQ(MOJO_RESULT_OK, ht.GetHandle(hv, &h));
|
| + EXPECT_EQ(dv, h.dispatcher.get());
|
|
|
| d = nullptr;
|
| - ASSERT_EQ(MOJO_RESULT_OK, ht.GetAndRemoveDispatcher(h, &d));
|
| - ASSERT_EQ(d.get(), dv);
|
| + ASSERT_EQ(MOJO_RESULT_OK, ht.GetAndRemoveDispatcher(hv, &d));
|
| + ASSERT_EQ(dv, d.get());
|
|
|
| EXPECT_EQ(MOJO_RESULT_OK, d->Close());
|
|
|
| - // We removed |h|, so it should no longer be valid.
|
| - d = nullptr;
|
| - EXPECT_EQ(MOJO_RESULT_INVALID_ARGUMENT, ht.GetDispatcher(h, &d));
|
| + // We removed |hv|, so it should no longer be valid.
|
| + h.reset();
|
| + EXPECT_EQ(MOJO_RESULT_INVALID_ARGUMENT, ht.GetHandle(hv, &h));
|
| }
|
|
|
| TEST(HandleTableTest, AddHandlePair) {
|
|
|