Remove base::SharedMemory::ShareToProcess.

content/browser/browser_child_process_host_impl.cc

Index: content/browser/browser_child_process_host_impl.cc
diff --git a/content/browser/browser_child_process_host_impl.cc b/content/browser/browser_child_process_host_impl.cc
index 8006a66890dfb0764e8da6dfeec20033b87745f1..3ddc8b5f021d7176faa6e95602036e05b9983453 100644
--- a/content/browser/browser_child_process_host_impl.cc
+++ b/content/browser/browser_child_process_host_impl.cc
@@ -512,9 +512,8 @@ void BrowserChildProcessHostImpl::CreateMetricsAllocator() {
 
 void BrowserChildProcessHostImpl::ShareMetricsAllocatorToProcess() {
   if (metrics_allocator_) {
-    base::SharedMemoryHandle shm_handle;
-    metrics_allocator_->shared_memory()->ShareToProcess(data_.handle,
-                                                        &shm_handle);
+    base::SharedMemoryHandle shm_handle =
+        metrics_allocator_->shared_memory()->handle().Duplicate();
     Send(new ChildProcessMsg_SetHistogramMemory(

[Nico, 2017/05/02 16:06:12]

Does sending a SharedMemoryHandle consume it? If not, why is the Duplicate()
needed?

[erikchen, 2017/05/02 19:00:51]

Yes, I've updated the documentation for Duplicate().

         shm_handle, metrics_allocator_->shared_memory()->mapped_size()));
   }