Remove base::SharedMemory::ShareToProcess.

media/gpu/video_encode_accelerator_unittest.cc

 // Copyright 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 <inttypes.h>
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
 #include <memory>
@@ skipping 1628 matching lines @@
 }
 
 void VEAClient::FeedEncoderWithOutput(base::SharedMemory* shm) {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (!has_encoder())
     return;
 
   if (state_ != CS_ENCODING)
     return;
 
-  base::SharedMemoryHandle dup_handle;
-  LOG_ASSERT(shm->ShareToProcess(base::GetCurrentProcessHandle(), &dup_handle));
+  base::SharedMemoryHandle dup_handle = shm->handle().Duplicate();
+  LOG_ASSERT(dup_handle.IsValid());
 
+  // TODO(erikchen): This may leak the SharedMemoryHandle.
+  // https://crbug.com/640840.
   BitstreamBuffer bitstream_buffer(next_output_buffer_id_++, dup_handle,
                                    output_buffer_size_);
   LOG_ASSERT(output_buffers_at_client_
                  .insert(std::make_pair(bitstream_buffer.id(), shm))
                  .second);
 
   encode_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&VideoEncodeAccelerator::UseOutputBitstreamBuffer,
                  encoder_weak_factory_->GetWeakPtr(), bitstream_buffer));
@@ skipping 243 matching lines @@
     output_shms_.push_back(shm);
     FeedEncoderWithOutput(shm, output_size);
   }
 }
 
 void SimpleVEAClientBase::FeedEncoderWithOutput(base::SharedMemory* shm,
                                                 size_t output_size) {
   if (!has_encoder())
     return;
 
-  base::SharedMemoryHandle dup_handle;
-  LOG_ASSERT(shm->ShareToProcess(base::GetCurrentProcessHandle(), &dup_handle));
+  base::SharedMemoryHandle dup_handle = shm->handle().Duplicate();
+  LOG_ASSERT(dup_handle.IsValid());
 
   BitstreamBuffer bitstream_buffer(next_output_buffer_id_++, dup_handle,
                                    output_size);
   encoder_->UseOutputBitstreamBuffer(bitstream_buffer);
 }
 
 // This client is only used to make sure the encoder does not return an encoded
 // frame before getting any input.
 class VEANoInputClient : public SimpleVEAClientBase {
  public:
@@ skipping 451 matching lines @@
 
   media::g_env =
       reinterpret_cast<media::VideoEncodeAcceleratorTestEnvironment*>(
           testing::AddGlobalTestEnvironment(
               new media::VideoEncodeAcceleratorTestEnvironment(
                   std::move(test_stream_data), log_path, run_at_fps,
                   needs_encode_latency, verify_all_output)));
 
   return RUN_ALL_TESTS();
 }