Integrate VDA with WebRTC.

content/renderer/media/rtc_video_decoder_factory.cc

+// Copyright (c) 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_decoder_factory.h"
+
+namespace content {
+
+RTCVideoDecoderFactory::RTCVideoDecoderFactory(
+    const scoped_refptr<base::MessageLoopProxy>& chrome_loop_proxy,
+    const scoped_refptr<media::GpuVideoDecoder::Factories>& gpu_factories)
+  : chrome_loop_proxy_(chrome_loop_proxy),

[Ami GONE FROM CHROMIUM, 2013/04/26 00:42:05]

nit: +2 additional indent this and the next line

[wuchengli, 2013/05/08 15:58:56]

Done.

+    gpu_factories_(gpu_factories) {
+
+}
+
+webrtc::VideoDecoder* RTCVideoDecoderFactory::CreateVideoDecoder(
+    webrtc::VideoCodecType type) {
+  if (type == webrtc::kVideoCodecVP8) {

[Ami GONE FROM CHROMIUM, 2013/04/26 00:42:05]

why not also allow h264 here?

[Pawel Osciak, 2013/04/26 07:08:01]

This should be decided by each vda implementation ideally...

[wuchengli, 2013/05/08 15:58:56]

H264 is not in the enum. All enum values are kVideoCodecVP8, kVideoCodecI420,
kVideoCodecRED, kVideoCodecULPFEC, kVideoCodecGeneric, and kVideoCodecUnknown.

It looks like libjingle expects this method to return null if the codec is not
supported. Maybe we should call GpuVideoDecoder::Initialize here. However,
GpuVideoDecoder::Initialize has a DHECK about VideoDecoderConfig.size != 0. We
don't have size information here.

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

On 2013/04/26 07:08:01, posciak wrote:

+    scoped_refptr<media::VideoDecoder> gpu_video_decoder =
+        new media::GpuVideoDecoder(chrome_loop_proxy_, gpu_factories_);
+
+    return new RTCVideoDecoderBridge(gpu_video_decoder, chrome_loop_proxy_);
+  }
+  return NULL;
+}
+
+void RTCVideoDecoderFactory::DestroyVideoDecoder(
+    webrtc::VideoDecoder* decoder) {
+  delete decoder;
+}
+
+}  // namespace content