Integrate VDA with WebRTC.

content/browser/renderer_host/render_process_host_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.

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

Please update the CL description to point to any dependent CLs in
libjingle/webrtc.

[wuchengli, 2013/04/26 11:49:14]

I added the WebRTC CL, though the link may need to change after I correct the
base URL. Libjingle is in p10. Can we put internal url in the CL description?
On 2013/04/26 00:42:05, Ami Fischman wrote:

 // Use of this source code is governed by a BSD-style license that can be

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

FYI local preview doesn't use the codec (it simply renders captured frames
without encoding+decoding them) so it sounds like your decoder machinery isn't
getting kicked off.  Perhaps the problem is a lack of registration?

It might be instructive to look at the devtools JS console for the apprtc page
to see how far that gets.

Have you tried instrumenting the bridge & factory to see what gets
instantiated/called?

[wuchengli, 2013/04/26 11:49:14]

I pasted the log at https://p.prom.corp.google.com/2594062. The decoder was
running. Sometimes loopback started. But overall it was sluggish intermittently.
On 2013/04/26 00:42:05, Ami Fischman wrote:

 // found in the LICENSE file.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <vector>
@@ skipping 809 matching lines @@
     switches::kEnableBrowserPluginForAllViewTypes,
     switches::kEnableDCHECK,
     switches::kEnableDelegatedRenderer,
     switches::kDisableEncryptedMedia,
     switches::kEnableExperimentalWebKitFeatures,
     switches::kEnableFixedLayout,
     switches::kEnableDeferredImageDecoding,
     switches::kEnableGPUServiceLogging,
     switches::kEnableGPUClientLogging,
     switches::kEnableGpuClientTracing,
+    switches::kEnableWebRTCHWDecoding,

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

This is a strange place to drop this particular flag :)

[wuchengli, 2013/04/26 11:49:14]

I moved it after switches::kEnableVsyncNotification. Some items in this array
does not follow alphabetical order... Let me know if you think there's a better
place. I assume you don't mean to move this to another file.

     switches::kEnableGpuBenchmarking,
     switches::kEnableMemoryBenchmarking,
     switches::kEnableLogging,
     switches::kEnableVsyncNotification,
     switches::kDisableMediaSource,
     switches::kDisableRendererSideMixing,
     switches::kEnableStrictSiteIsolation,
     switches::kDisableFullScreen,
     switches::kEnableNewDialogStyle,
 #if defined(ENABLE_PLUGINS)
@@ skipping 879 matching lines @@
   TRACE_EVENT0("renderer_host",
                "RenderWidgetHostImpl::OnCompositorSurfaceBuffersSwappedNoHost");
   AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
   ack_params.sync_point = 0;
   RenderWidgetHostImpl::AcknowledgeBufferPresent(route_id,
                                                  gpu_process_host_id,
                                                  ack_params);
 }
 
 }  // namespace content