To dropp the decode done callback in RTCVideoDecoderBridgeTv.

content/renderer/media/rtc_video_decoder_bridge_tv.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_decoder_bridge_tv.h"
 
 #include <queue>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 73 matching lines @@
   }
   // |input_image_| may be destroyed after this call, so we make a copy of the
   // buffer so that we can queue the buffer asynchronously.
   scoped_refptr<media::DecoderBuffer> buffer =
       media::DecoderBuffer::CopyFrom(input_image._buffer, input_image._length);
   if (render_time_ms != -1) {
     buffer->set_timestamp(base::TimeDelta::FromMilliseconds(
         render_time_ms - timestamp_offset_millis_));
   }
 
-  factory_->QueueBuffer(
-      buffer,
-      base::Bind(&RTCVideoDecoderBridgeTv::RunDecodeCompleteCallback,
-                 decode_complete_callback_,
-                 input_image._timeStamp,
-                 size_),
-      new_size);
+  factory_->QueueBuffer(buffer, new_size);
+
+  // WebRTC measures the decoding time and control the timing of the Decode()
+  // calls. In GoogleTV's implementation, we let WebRTC think decoding is done
+  // here so that we can get frames constantly.
+  RunDecodeCompleteCallback(input_image._timeStamp);
 
   return WEBRTC_VIDEO_CODEC_OK;
 }
 
 int32_t RTCVideoDecoderBridgeTv::RegisterDecodeCompleteCallback(
     webrtc::DecodedImageCallback* callback) {
   decode_complete_callback_ = callback;
   return WEBRTC_VIDEO_CODEC_OK;
 }
 
 int32_t RTCVideoDecoderBridgeTv::Release() {
   is_initialized_ = false;
   return WEBRTC_VIDEO_CODEC_OK;
 }
 
 int32_t RTCVideoDecoderBridgeTv::Reset() {
   first_frame_ = true;
   return WEBRTC_VIDEO_CODEC_OK;
 }
 
-// static
-void RTCVideoDecoderBridgeTv::RunDecodeCompleteCallback(
-    webrtc::DecodedImageCallback* callback,
-    int64_t timestamp,
-    gfx::Size size) {
+void RTCVideoDecoderBridgeTv::RunDecodeCompleteCallback(int64_t timestamp) {

[wonsik, 2013/07/29 02:33:28]

Since this is used only in RTCVideoDecoderBridgeTv::Decode, you can merge this
method into Decode.

[dwkang1, 2013/07/29 02:55:35]

Done.

   // We call the decode complete callback function to notify libjingle that
   // decoding is finished. In addition, this also reports back to libjingle that
   // the particular video frame with |timestamp| is correctly rendered to
   // libjingle, so that it can generate proper stats.
   webrtc::I420VideoFrame dummy_video_frame;
-  int half_width = (size.width() + 1) / 2;
+  int half_width = (size_.width() + 1) / 2;
   dummy_video_frame.CreateEmptyFrame(
-      size.width(), size.height(), size.width(), half_width, half_width);
+      size_.width(), size_.height(), size_.width(), half_width, half_width);
   dummy_video_frame.set_timestamp(timestamp);
-  callback->Decoded(dummy_video_frame);
+  decode_complete_callback_->Decoded(dummy_video_frame);
 }
 
 }  // namespace content