Report VideoDecoder status through ReadCB instead of through FilterHost.

content/renderer/media/rtc_video_decoder.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/renderer/media/rtc_video_decoder.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/message_loop.h"
 #include "media/base/demuxer.h"
-#include "media/base/filter_host.h"
 #include "media/base/filters.h"
 #include "media/base/limits.h"
 #include "media/base/video_frame.h"
 #include "media/base/video_util.h"
 #include "third_party/libjingle/source/talk/session/phone/videoframe.h"
 
 using media::CopyUPlane;
 using media::CopyVPlane;
 using media::CopyYPlane;
 using media::DemuxerStream;
@@ skipping 76 matching lines @@
     base::AutoLock auto_lock(lock_);
     if (!read_cb_.is_null()) {
       std::swap(read_cb, read_cb_);
     }
   }
 
   if (!read_cb.is_null()) {
     scoped_refptr<media::VideoFrame> video_frame =
         media::VideoFrame::CreateBlackFrame(visible_size_.width(),
                                             visible_size_.height());
-    read_cb.Run(video_frame);
+    read_cb.Run(kOk, video_frame);
   }
 
   VideoDecoder::Flush(callback);
 }
 
 void RTCVideoDecoder::Stop(const base::Closure& callback) {
   if (MessageLoop::current() != message_loop_) {
     message_loop_->PostTask(FROM_HERE,
                             base::Bind(&RTCVideoDecoder::Stop,
                                        this, callback));
@@ skipping 33 matching lines @@
   read_cb_ = callback;
 }
 
 const gfx::Size& RTCVideoDecoder::natural_size() {
   // TODO(vrk): Return natural size when aspect ratio support is implemented.
   return visible_size_;
 }
 
 bool RTCVideoDecoder::SetSize(int width, int height, int reserved) {
   visible_size_.SetSize(width, height);
-
-  // TODO(vrk): Provide natural size when aspect ratio support is implemented.
-
-  // TODO(xhwang) host() can be NULL after r128289.  Remove this check when
-  // it is no longer needed.
-  if (host())
-    host()->SetNaturalVideoSize(visible_size_);
   return true;
 }
 
 // TODO(wjia): this function can be split into two parts so that the |lock_|
 // can be removed.
 // First creates media::VideoFrame, then post a task onto |message_loop_|
 // to deliver that frame.
 bool RTCVideoDecoder::RenderFrame(const cricket::VideoFrame* frame) {
   // Called from libjingle thread.
   DCHECK(frame);
@@ skipping 36 matching lines @@
   // Aspect ratio unsupported; DCHECK when there are non-square pixels.
   DCHECK_EQ(frame->GetPixelWidth(), 1u);
   DCHECK_EQ(frame->GetPixelHeight(), 1u);
 
   int y_rows = frame->GetHeight();
   int uv_rows = frame->GetHeight() / 2;  // YV12 format.
   CopyYPlane(frame->GetYPlane(), frame->GetYPitch(), y_rows, video_frame);
   CopyUPlane(frame->GetUPlane(), frame->GetUPitch(), uv_rows, video_frame);
   CopyVPlane(frame->GetVPlane(), frame->GetVPitch(), uv_rows, video_frame);
 
-  read_cb.Run(video_frame);
+  read_cb.Run(kOk, video_frame);
   return true;
 }