Report VideoDecoder status through ReadCB instead of through FilterHost.

media/base/filters.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 "media/base/filters.h"
 
 #include "base/logging.h"
 
 namespace media {
 
@@ skipping 56 matching lines @@
 
 void VideoDecoder::Pause(const base::Closure& /* callback */) {
   LOG(FATAL) << "VideoDecoder::Pause is not supposed to be called.";
 }
 
 void VideoDecoder::Seek(base::TimeDelta /* time */,
                         const PipelineStatusCB& /* callback */) {
   LOG(FATAL) << "VideoDecoder::Seek is not supposed to be called.";
 }
 
+FilterHost* VideoDecoder::host(){
+  LOG(FATAL) << "VideoDecoder::host is not supposed to be called.";

[scherkus (not reviewing), 2012/04/27 23:28:33]

add TODO w/ crbug entry pointing at removing Filter

[xhwang, 2012/04/27 23:32:25]

TODO is in filter.h together with Pause()/Seek()...

+  return NULL;

[Ami GONE FROM CHROMIUM, 2012/04/27 23:32:05]

I'm surprised this is necessary (since LOG(FATAL) doesn't return).

+}
+
 bool VideoDecoder::HasAlpha() const {
   return false;
 }
 
 void VideoDecoder::PrepareForShutdownHack() {}
 
 }  // namespace media