Remove base::SharedMemory::ShareToProcess.

components/display_compositor/host_shared_bitmap_manager_unittest.cc

 // Copyright (c) 2013 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 <stddef.h>
 #include <string.h>
 
 #include "components/display_compositor/host_shared_bitmap_manager.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace display_compositor {
 namespace {
 
 class HostSharedBitmapManagerTest : public testing::Test {
  protected:
   void SetUp() override { manager_.reset(new HostSharedBitmapManager()); }
   std::unique_ptr<HostSharedBitmapManager> manager_;
 };
 
 TEST_F(HostSharedBitmapManagerTest, TestCreate) {
   gfx::Size bitmap_size(1, 1);
   size_t size_in_bytes;
   EXPECT_TRUE(cc::SharedBitmap::SizeInBytes(bitmap_size, &size_in_bytes));
   std::unique_ptr<base::SharedMemory> bitmap(new base::SharedMemory());
   bitmap->CreateAndMapAnonymous(size_in_bytes);
   memset(bitmap->memory(), 0xff, size_in_bytes);
   cc::SharedBitmapId id = cc::SharedBitmap::GenerateId();
 
   HostSharedBitmapManagerClient client(manager_.get());
-  base::SharedMemoryHandle handle;
-  bitmap->ShareToProcess(base::GetCurrentProcessHandle(), &handle);
+  base::SharedMemoryHandle handle = bitmap->handle().Duplicate();
   client.ChildAllocatedSharedBitmap(size_in_bytes, handle, id);

[Nico, 2017/05/02 20:08:00]

I did check that this does assume ownership.

 
   std::unique_ptr<cc::SharedBitmap> large_bitmap;
   large_bitmap = manager_->GetSharedBitmapFromId(gfx::Size(1024, 1024), id);
   EXPECT_TRUE(large_bitmap.get() == NULL);
 
   std::unique_ptr<cc::SharedBitmap> very_large_bitmap;
   very_large_bitmap =
       manager_->GetSharedBitmapFromId(gfx::Size(1, (1 << 30) | 1), id);
   EXPECT_TRUE(very_large_bitmap.get() == NULL);
 
@@ skipping 35 matching lines @@
 
 TEST_F(HostSharedBitmapManagerTest, RemoveProcess) {
   gfx::Size bitmap_size(1, 1);
   size_t size_in_bytes;
   EXPECT_TRUE(cc::SharedBitmap::SizeInBytes(bitmap_size, &size_in_bytes));
   std::unique_ptr<base::SharedMemory> bitmap(new base::SharedMemory());
   bitmap->CreateAndMapAnonymous(size_in_bytes);
   memset(bitmap->memory(), 0xff, size_in_bytes);
   cc::SharedBitmapId id = cc::SharedBitmap::GenerateId();
 
-  base::SharedMemoryHandle handle;
   std::unique_ptr<HostSharedBitmapManagerClient> client(
       new HostSharedBitmapManagerClient(manager_.get()));
-  bitmap->ShareToProcess(base::GetCurrentProcessHandle(), &handle);
+  base::SharedMemoryHandle handle = bitmap->handle().Duplicate();
   client->ChildAllocatedSharedBitmap(size_in_bytes, handle, id);
 
   std::unique_ptr<cc::SharedBitmap> shared_bitmap;
   shared_bitmap = manager_->GetSharedBitmapFromId(bitmap_size, id);
   ASSERT_TRUE(shared_bitmap.get() != NULL);
 
   EXPECT_EQ(1u, manager_->AllocatedBitmapCount());
   client.reset();
   EXPECT_EQ(0u, manager_->AllocatedBitmapCount());
 
   std::unique_ptr<cc::SharedBitmap> shared_bitmap2;
   shared_bitmap2 = manager_->GetSharedBitmapFromId(bitmap_size, id);
   EXPECT_TRUE(shared_bitmap2.get() == NULL);
   EXPECT_EQ(memcmp(shared_bitmap->pixels(), bitmap->memory(), size_in_bytes),
             0);
 
   shared_bitmap.reset();
 }
 
 TEST_F(HostSharedBitmapManagerTest, AddDuplicate) {
   gfx::Size bitmap_size(1, 1);
   size_t size_in_bytes;
   EXPECT_TRUE(cc::SharedBitmap::SizeInBytes(bitmap_size, &size_in_bytes));
   std::unique_ptr<base::SharedMemory> bitmap(new base::SharedMemory());
   bitmap->CreateAndMapAnonymous(size_in_bytes);
   memset(bitmap->memory(), 0xff, size_in_bytes);
   cc::SharedBitmapId id = cc::SharedBitmap::GenerateId();
   HostSharedBitmapManagerClient client(manager_.get());
 
-  base::SharedMemoryHandle handle;
-  bitmap->ShareToProcess(base::GetCurrentProcessHandle(), &handle);
+  base::SharedMemoryHandle handle = bitmap->handle().Duplicate();
   client.ChildAllocatedSharedBitmap(size_in_bytes, handle, id);
 
   std::unique_ptr<base::SharedMemory> bitmap2(new base::SharedMemory());
   bitmap2->CreateAndMapAnonymous(size_in_bytes);
   memset(bitmap2->memory(), 0x00, size_in_bytes);
 
   client.ChildAllocatedSharedBitmap(size_in_bytes, bitmap2->handle(), id);
 
   std::unique_ptr<cc::SharedBitmap> shared_bitmap;
   shared_bitmap = manager_->GetSharedBitmapFromId(bitmap_size, id);
   ASSERT_TRUE(shared_bitmap.get() != NULL);
   EXPECT_EQ(memcmp(shared_bitmap->pixels(), bitmap->memory(), size_in_bytes),
             0);
   client.DidDeleteSharedBitmap(id);
 }
 
 }  // namespace
 }  // namespace display_compositor