Cleanup of video capture into GpuMemoryBuffer

content/browser/renderer_host/media/video_capture_controller.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/media/video_capture_controller.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <map>
@@ skipping 201 matching lines @@
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   DVLOG(1) << "VideoCaptureController::AddClient() -- id=" << id
            << ", session_id=" << session_id
            << ", params.requested_format="
            << media::VideoCaptureFormat::ToString(params.requested_format);
 
   // Check that requested VideoCaptureParams are valid and supported.  If not,
   // report an error immediately and punt.
   if (!params.IsValid() ||
       params.requested_format.pixel_format != media::PIXEL_FORMAT_I420 ||
-      (params.requested_format.pixel_storage != media::PIXEL_STORAGE_CPU &&
-       params.requested_format.pixel_storage !=
-           media::PIXEL_STORAGE_GPUMEMORYBUFFER)) {
+      params.requested_format.pixel_storage != media::PIXEL_STORAGE_CPU) {
     // Crash in debug builds since the renderer should not have asked for
     // invalid or unsupported parameters.
     LOG(DFATAL) << "Invalid or unsupported video capture parameters requested: "
                 << media::VideoCaptureFormat::ToString(params.requested_format);
     event_handler->OnError(id);
     return;
   }
 
   // If this is the first client added to the controller, cache the parameters.
   if (controller_clients_.empty())
@@ skipping 278 matching lines @@
     const scoped_refptr<media::VideoFrame>& frame) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   const int buffer_id = buffer->id();
 
   switch (frame->storage_type()) {
     case media::VideoFrame::STORAGE_GPU_MEMORY_BUFFERS: {
       std::vector<gfx::GpuMemoryBufferHandle> handles;
       const size_t num_planes = media::VideoFrame::NumPlanes(frame->format());
       for (size_t i = 0; i < num_planes; ++i) {
         gfx::GpuMemoryBufferHandle remote_handle;
-        buffer_pool_->ShareToProcess2(
-            buffer_id, i, client->render_process_handle, &remote_handle);
         handles.push_back(remote_handle);
       }
       client->event_handler->OnBufferCreated2(client->controller_id, handles,
                                               buffer->dimensions(), buffer_id);
       break;
     }
     case media::VideoFrame::STORAGE_SHMEM: {
       base::SharedMemoryHandle remote_handle;
       buffer_pool_->ShareToProcess(buffer_id, client->render_process_handle,
                                    &remote_handle);
@@ skipping 23 matching lines @@
     int session_id,
     const ControllerClients& clients) {
   for (const auto& client : clients) {
     if (client->session_id == session_id)
       return client.get();
   }
   return nullptr;
 }
 
 }  // namespace content