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

mojo/edk/system/remote_data_pipe_impl_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file tests both |RemoteProducerDataPipeImpl| and
 // |RemoteConsumerDataPipeImpl|.
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 216 matching lines @@
                                          &read_handles, &read_num_handles,
                                          MOJO_READ_MESSAGE_FLAG_NONE));
   EXPECT_EQ(0u, static_cast<size_t>(read_buffer_size));
   EXPECT_EQ(1u, read_handles.size());
   EXPECT_EQ(1u, read_num_handles);
   ASSERT_TRUE(read_handles[0]);
   EXPECT_TRUE(read_handles[0].dispatcher->HasOneRef());
 
   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()));
   read_handles.clear();
 
   waiter.Init();
   hss = HandleSignalsState();
   MojoResult result =
       consumer->AddAwakable(&waiter, MOJO_HANDLE_SIGNAL_READABLE, 456, &hss);
   if (result == MOJO_RESULT_OK) {
@@ skipping 101 matching lines @@
                                          &read_handles, &read_num_handles,
                                          MOJO_READ_MESSAGE_FLAG_NONE));
   EXPECT_EQ(0u, static_cast<size_t>(read_buffer_size));
   EXPECT_EQ(1u, read_handles.size());
   EXPECT_EQ(1u, read_num_handles);
   ASSERT_TRUE(read_handles[0]);
   EXPECT_TRUE(read_handles[0].dispatcher->HasOneRef());
 
   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()));
   read_handles.clear();
 
   // Now actually write the data, complete the two-phase write, and close the
   // producer.
   *static_cast<int32_t*>(write_ptr) = 123456;
   EXPECT_EQ(MOJO_RESULT_OK, dp->ProducerEndWriteData(
                                 static_cast<uint32_t>(1u * sizeof(int32_t))));
@@ skipping 105 matching lines @@
                                          &read_handles, &read_num_handles,
                                          MOJO_READ_MESSAGE_FLAG_NONE));
   EXPECT_EQ(0u, static_cast<size_t>(read_buffer_size));
   EXPECT_EQ(1u, read_handles.size());
   EXPECT_EQ(1u, read_num_handles);
   ASSERT_TRUE(read_handles[0]);
   EXPECT_TRUE(read_handles[0].dispatcher->HasOneRef());
 
   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()));
   read_handles.clear();
 
   // Now actually write the data, complete the two-phase write, and close the
   // producer.
   *static_cast<int32_t*>(write_ptr) = 789012;
   EXPECT_EQ(MOJO_RESULT_OK, dp->ProducerEndWriteData(
                                 static_cast<uint32_t>(1u * sizeof(int32_t))));
@@ skipping 29 matching lines @@
   EXPECT_EQ(123456, elements[0]);
   EXPECT_EQ(789012, elements[1]);
   EXPECT_EQ(0, elements[2]);
 
   consumer->Close();
 }
 
 }  // namespace
 }  // namespace system
 }  // namespace mojo