Replace base::SharedMemory read-only methods with GetReadOnlyHandle.

base/memory/shared_memory_mac_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.
 
 #include <mach/mach.h>
 #include <mach/mach_vm.h>
 #include <servers/bootstrap.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 376 matching lines @@
     SharedMemory shared_memory2(shm2, true);
     shared_memory2.Map(s_memory_size);
 
     ASSERT_EQ(0, memcmp(shared_memory->memory(), shared_memory2.memory(),
                         s_memory_size));
   }
 
   EXPECT_EQ(active_name_count, GetActiveNameCount());
 }
 
-// Tests that the method ShareReadOnlyToProcess() creates a memory object that
+// Tests that the method GetReadOnlyHandle() creates a memory object that
 // is read only.
 TEST_F(SharedMemoryMacMultiProcessTest, MachShareToProcessReadonly) {
   std::unique_ptr<SharedMemory> shared_memory(
       CreateSharedMemory(s_memory_size));
 
   // Check the protection levels.
   int current_prot, max_prot;
   ASSERT_TRUE(GetProtections(shared_memory->memory(),
                              shared_memory->mapped_size(), &current_prot,
                              &max_prot));
   ASSERT_EQ(VM_PROT_READ | VM_PROT_WRITE, current_prot);
   ASSERT_EQ(VM_PROT_READ | VM_PROT_WRITE, max_prot);
 
   // Make a new memory object.
-  SharedMemoryHandle shm2;
-  ASSERT_TRUE(shared_memory->ShareReadOnlyToProcess(GetCurrentProcId(), &shm2));
+  SharedMemoryHandle shm2 = shared_memory->GetReadOnlyHandle();
   ASSERT_TRUE(shm2.IsValid());
 
   // Mapping with |readonly| set to |false| should fail.
   SharedMemory shared_memory2(shm2, false);
   shared_memory2.Map(s_memory_size);
   ASSERT_EQ(nullptr, shared_memory2.memory());
 
   // Now trying mapping with |readonly| set to |true|.
   SharedMemory shared_memory3(shm2.Duplicate(), true);
   shared_memory3.Map(s_memory_size);
   ASSERT_NE(nullptr, shared_memory3.memory());
 
   // Check the protection levels.
   ASSERT_TRUE(GetProtections(shared_memory3.memory(),
                              shared_memory3.mapped_size(), &current_prot,
                              &max_prot));
   ASSERT_EQ(VM_PROT_READ, current_prot);
   ASSERT_EQ(VM_PROT_READ, max_prot);
 
   // The memory should still be readonly, since the underlying memory object
   // is readonly.
   ASSERT_DEATH(memset(shared_memory2.memory(), 'b', s_memory_size), "");
 }
 
-// Tests that the method ShareReadOnlyToProcess() doesn't leak.
+// Tests that the method GetReadOnlyHandle() doesn't leak.
 TEST_F(SharedMemoryMacMultiProcessTest, MachShareToProcessReadonlyLeak) {
   mach_msg_type_number_t active_name_count = GetActiveNameCount();
 
   {
     std::unique_ptr<SharedMemory> shared_memory(
         CreateSharedMemory(s_memory_size));
 
-    SharedMemoryHandle shm2;
-    ASSERT_TRUE(
-        shared_memory->ShareReadOnlyToProcess(GetCurrentProcId(), &shm2));
+    SharedMemoryHandle shm2 = shared_memory->GetReadOnlyHandle();
     ASSERT_TRUE(shm2.IsValid());
 
     // Intentionally map with |readonly| set to |false|.
     SharedMemory shared_memory2(shm2, false);
     shared_memory2.Map(s_memory_size);
   }
 
   EXPECT_EQ(active_name_count, GetActiveNameCount());
 }
 
 }  //  namespace base