EDK: Check that handle rights are preserved in various tests.

mojo/edk/system/remote_data_pipe_impl_unittest.cc

Index: mojo/edk/system/remote_data_pipe_impl_unittest.cc
diff --git a/mojo/edk/system/remote_data_pipe_impl_unittest.cc b/mojo/edk/system/remote_data_pipe_impl_unittest.cc
index 24df824482d16580f05fc6fbc34c0ff750a42403..86c54ce171e24d5a438275dbe8c443d99910ee25 100644
--- a/mojo/edk/system/remote_data_pipe_impl_unittest.cc
+++ b/mojo/edk/system/remote_data_pipe_impl_unittest.cc
@@ -234,7 +234,8 @@ TEST_F(RemoteDataPipeImplTest, SendConsumerWithClosedProducer) {
 
   EXPECT_EQ(Dispatcher::Type::DATA_PIPE_CONSUMER,
             read_handles[0].dispatcher->GetType());
-  // TODO(vtl): Also check the rights here once they're actually preserved?
+  EXPECT_EQ(DataPipeConsumerDispatcher::kDefaultHandleRights,
+            read_handles[0].rights);
   consumer = RefPtr<DataPipeConsumerDispatcher>(
       static_cast<DataPipeConsumerDispatcher*>(
           read_handles[0].dispatcher.get()));
@@ -356,7 +357,8 @@ TEST_F(RemoteDataPipeImplTest, SendConsumerDuringTwoPhaseWrite) {
 
   EXPECT_EQ(Dispatcher::Type::DATA_PIPE_CONSUMER,
             read_handles[0].dispatcher->GetType());
-  // TODO(vtl): Also check the rights here once they're actually preserved?
+  EXPECT_EQ(DataPipeConsumerDispatcher::kDefaultHandleRights,
+            read_handles[0].rights);
   consumer = RefPtr<DataPipeConsumerDispatcher>(
       static_cast<DataPipeConsumerDispatcher*>(
           read_handles[0].dispatcher.get()));
@@ -482,7 +484,8 @@ TEST_F(RemoteDataPipeImplTest, SendConsumerDuringSecondTwoPhaseWrite) {
 
   EXPECT_EQ(Dispatcher::Type::DATA_PIPE_CONSUMER,
             read_handles[0].dispatcher->GetType());
-  // TODO(vtl): Also check the rights here once they're actually preserved?
+  EXPECT_EQ(DataPipeConsumerDispatcher::kDefaultHandleRights,
+            read_handles[0].rights);
   consumer = RefPtr<DataPipeConsumerDispatcher>(
       static_cast<DataPipeConsumerDispatcher*>(
           read_handles[0].dispatcher.get()));