Remove base::SharedMemory::ShareToProcess.

gpu/ipc/service/gpu_channel_test_common.cc

 // Copyright (c) 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 "gpu/ipc/service/gpu_channel_test_common.h"
 
 #include "base/memory/ptr_util.h"
 #include "base/memory/shared_memory.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 134 matching lines @@
     deserializer->SerializeOutputParameters(*reply_msg);
 
     delete deserializer;
   }
 
   sink->ClearMessages();
 
   delete msg;
 }
 
 base::SharedMemoryHandle GpuChannelTestCommon::GetSharedHandle() {

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

It's weird to have things transferring ownership start with just Get(). Maybe
that's Cocoa speaking, but it seems like this is just asking for bugs.

[erikchen, 2017/05/02 21:33:10]

This is a non-virtual method defined in a test class. *shrug*.

   base::SharedMemory shared_memory;
   shared_memory.CreateAnonymous(10);
-  base::SharedMemoryHandle shmem_handle;
-  shared_memory.ShareToProcess(base::GetCurrentProcessHandle(), &shmem_handle);
-  return shmem_handle;
+  return shared_memory.handle().Duplicate();
 }
 
 }  // namespace gpu