Passing Native Texture backed Video Frame from Renderer to GPU process

content/renderer/media/rtc_video_encoder.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 "content/renderer/media/rtc_video_encoder.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/memory/scoped_vector.h"
 #include "base/metrics/histogram.h"
 #include "base/rand_util.h"
 #include "base/single_thread_task_runner.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/thread_task_runner_handle.h"
+#include "media/base/bind_to_current_loop.h"
 #include "media/base/bitstream_buffer.h"
 #include "media/base/video_frame.h"
 #include "media/base/video_util.h"
 #include "media/filters/h264_parser.h"
 #include "media/renderers/gpu_video_accelerator_factories.h"
 #include "media/video/video_encode_accelerator.h"
 #include "third_party/libyuv/include/libyuv.h"
 #include "third_party/webrtc/system_wrappers/interface/tick_util.h"
 
 #define NOTIFY_ERROR(x)                             \
@@ skipping 458 matching lines @@
   bool next_frame_keyframe = input_next_frame_keyframe_;
   input_next_frame_ = NULL;
   input_next_frame_keyframe_ = false;
 
   if (!video_encoder_) {
     SignalAsyncWaiter(WEBRTC_VIDEO_CODEC_ERROR);
     return;
   }
 
   const int index = input_buffers_free_.back();
+
+  if (next_frame->native_handle()) {
+    scoped_refptr<media::VideoFrame> frame(
+        static_cast<media::VideoFrame*>(next_frame->native_handle()));
+    frame->AddDestructionObserver(media::BindToCurrentLoop(
+        base::Bind(&RTCVideoEncoder::Impl::EncodeFrameFinished, this, index)));
+    video_encoder_->Encode(frame, next_frame_keyframe);

[Pawel Osciak, 2015/06/03 10:28:56]

Could we please not duplicate these lines, but instead do an if/else for
native/shm?

[emircan, 2015/06/03 20:56:12]

Done. 

I am preparing a followup CL for RTCVideoEncoder, such that it doesn't need to
rotate ShMem input buffers for native textures (Item 4 in my followup CLs list).
Things will be separated and cleaner there. 

+    input_buffers_free_.pop_back();
+    SignalAsyncWaiter(WEBRTC_VIDEO_CODEC_OK);
+    return;
+  }
+
   base::SharedMemory* input_buffer = input_buffers_[index];
   scoped_refptr<media::VideoFrame> frame =
       media::VideoFrame::WrapExternalSharedMemory(
           media::VideoFrame::I420,
           input_frame_coded_size_,
           gfx::Rect(input_visible_size_),
           input_visible_size_,
           reinterpret_cast<uint8*>(input_buffer->memory()),
           input_buffer->mapped_size(),
           input_buffer->handle(),
@@ skipping 275 matching lines @@
   UMA_HISTOGRAM_BOOLEAN("Media.RTCVideoEncoderInitEncodeSuccess",
                         init_retval == WEBRTC_VIDEO_CODEC_OK);
   if (init_retval == WEBRTC_VIDEO_CODEC_OK) {
     UMA_HISTOGRAM_ENUMERATION("Media.RTCVideoEncoderProfile",
                               profile,
                               media::VIDEO_CODEC_PROFILE_MAX + 1);
   }
 }
 
 }  // namespace content