Delete MockDataSource, remove IsLocalSource & IsSeekable from Demuxer/Pipeline.

MockDataSource was completely unused, and Is{LocalSource,Seekable} being in the
Demuxer is leftover craxy from the DataSourceFactory / DemuxerFactory /
DownloadRateMonitor days, may they forever RIP.

BUG=120426


Review URL: https://chromiumcodereview.appspot.com/10378100

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136679 0039d316-1c4b-4281-b951-d872f2087c98

6 files changed, 1 insertions, 110 deletions

diff --git a/media/base/demuxer.h b/media/base/demuxer.h
index 1e913d4..a8cf50b 100644
--- a/media/base/demuxer.h
+++ b/media/base/demuxer.h
@@ -70,15 +70,6 @@ class MEDIA_EXPORT Demuxer : public base::RefCountedThreadSafe<Demuxer> {
   // approximated. Returns 0 if it is unknown.
   virtual int GetBitrate() = 0;
 
-  // Returns true if the source is from a local file or stream (such as a
-  // webcam stream), false otherwise.
-  //
-  // TODO(scherkus): See http://crbug.com/120426 on why we should remove this.
-  virtual bool IsLocalSource() = 0;
-
-  // Returns true if seeking is possible; false otherwise.
-  virtual bool IsSeekable() = 0;
-
  protected:
   friend class base::RefCountedThreadSafe<Demuxer>;
   virtual ~Demuxer();
diff --git a/media/base/mock_filters.cc b/media/base/mock_filters.cc
index 00347cd..a5977a4 100644
--- a/media/base/mock_filters.cc
+++ b/media/base/mock_filters.cc
@@ -15,29 +15,6 @@ using ::testing::Return;
 
 namespace media {
 
-MockDataSource::MockDataSource()
-    : total_bytes_(-1),
-      buffered_bytes_(-1) {
-}
-
-MockDataSource::~MockDataSource() {}
-
-void MockDataSource::set_host(DataSourceHost* data_source_host) {
-  DataSource::set_host(data_source_host);
-
-  if (total_bytes_ > 0)
-    host()->SetTotalBytes(total_bytes_);
-
-  if (buffered_bytes_ > 0)
-    host()->SetBufferedBytes(buffered_bytes_);
-}
-
-void MockDataSource::SetTotalAndBufferedBytes(int64 total_bytes,
-                                              int64 buffered_bytes) {
-  total_bytes_ = total_bytes;
-  buffered_bytes_ = buffered_bytes;
-}
-
 MockDemuxer::MockDemuxer() {}
 
 MockDemuxer::~MockDemuxer() {}
diff --git a/media/base/mock_filters.h b/media/base/mock_filters.h
index a5060d3..32b2333 100644
--- a/media/base/mock_filters.h
+++ b/media/base/mock_filters.h
@@ -3,8 +3,7 @@
 // found in the LICENSE file.
 //
 // A new breed of mock media filters, this time using gmock!  Feel free to add
-// actions if you need interesting side-effects (i.e., copying data to the
-// buffer passed into MockDataSource::Read()).
+// actions if you need interesting side-effects.
 //
 // Don't forget you can use StrictMock<> and NiceMock<> if you want the mock
 // filters to fail the test or do nothing when an unexpected method is called.
@@ -70,38 +69,6 @@ class MockFilter : public Filter {
   DISALLOW_COPY_AND_ASSIGN(MockFilter);
 };
 
-class MockDataSource : public DataSource {
- public:
-  MockDataSource();
-
-  virtual void set_host(DataSourceHost* data_source_host);
-
-  MOCK_METHOD1(Stop, void(const base::Closure& callback));
-  MOCK_METHOD1(SetPlaybackRate, void(float playback_rate));
-  MOCK_METHOD2(Seek, void(base::TimeDelta time, const PipelineStatusCB& cb));
-  MOCK_METHOD0(OnAudioRendererDisabled, void());
-
-  // DataSource implementation.
-  MOCK_METHOD4(Read, void(int64 position, int size, uint8* data,
-                          const DataSource::ReadCB& callback));
-  MOCK_METHOD1(GetSize, bool(int64* size_out));
-  MOCK_METHOD1(SetBitrate, void(int bitrate));
-  MOCK_METHOD0(IsStreaming, bool());
-
-  // Sets the TotalBytes & BufferedBytes values to be sent to host() when
-  // the set_host() is called.
-  void SetTotalAndBufferedBytes(int64 total_bytes, int64 buffered_bytes);
-
- protected:
-  virtual ~MockDataSource();
-
- private:
-  int64 total_bytes_;
-  int64 buffered_bytes_;
-
-  DISALLOW_COPY_AND_ASSIGN(MockDataSource);
-};
-
 class MockDemuxer : public Demuxer {
  public:
   MockDemuxer();
@@ -115,8 +82,6 @@ class MockDemuxer : public Demuxer {
   MOCK_METHOD1(GetStream, scoped_refptr<DemuxerStream>(DemuxerStream::Type));
   MOCK_CONST_METHOD0(GetStartTime, base::TimeDelta());
   MOCK_METHOD0(GetBitrate, int());
-  MOCK_METHOD0(IsLocalSource, bool());
-  MOCK_METHOD0(IsSeekable, bool());
 
  protected:
   virtual ~MockDemuxer();
diff --git a/media/base/pipeline.cc b/media/base/pipeline.cc
index 260f04d..623cecf 100644
--- a/media/base/pipeline.cc
+++ b/media/base/pipeline.cc
@@ -223,16 +223,6 @@ void Pipeline::GetNaturalVideoSize(gfx::Size* out_size) const {
   *out_size = natural_size_;
 }
 
-bool Pipeline::IsStreaming() const {
-  base::AutoLock auto_lock(lock_);
-  return demuxer_ && !demuxer_->IsSeekable();
-}
-
-bool Pipeline::IsLocalSource() const {
-  base::AutoLock auto_lock(lock_);
-  return demuxer_ && demuxer_->IsLocalSource();
-}
-
 PipelineStatistics Pipeline::GetStatistics() const {
   base::AutoLock auto_lock(lock_);
   return statistics_;
diff --git a/media/base/pipeline.h b/media/base/pipeline.h
index 699b5e2..73359c2 100644
--- a/media/base/pipeline.h
+++ b/media/base/pipeline.h
@@ -230,14 +230,6 @@ class MEDIA_EXPORT Pipeline
   // be 0.
   void GetNaturalVideoSize(gfx::Size* out_size) const;
 
-  // If this method returns true, that means the data source is a streaming
-  // data source. Seeking may not be possible.
-  bool IsStreaming() const;
-
-  // If this method returns true, that means the data source is local and
-  // the network is not needed.
-  bool IsLocalSource() const;
-
   // Gets the current pipeline statistics.
   PipelineStatistics GetStatistics() const;
 
diff --git a/media/base/pipeline_unittest.cc b/media/base/pipeline_unittest.cc
index 56d39ff..888b962 100644
--- a/media/base/pipeline_unittest.cc
+++ b/media/base/pipeline_unittest.cc
@@ -36,8 +36,6 @@ namespace media {
 static const int kTotalBytes = 1024;
 static const int kBufferedBytes = 1024;
 static const int kBitrate = 1234;
-static const bool kLocalSource = false;
-static const bool kSeekable = true;
 
 ACTION_P(InitializeDemuxerWithError, error) {
   arg1.Run(error);
@@ -117,10 +115,6 @@ class PipelineTest : public ::testing::Test {
     // Demuxer properties.
     EXPECT_CALL(*mocks_->demuxer(), GetBitrate())
         .WillRepeatedly(Return(kBitrate));
-    EXPECT_CALL(*mocks_->demuxer(), IsLocalSource())
-        .WillRepeatedly(Return(kLocalSource));
-    EXPECT_CALL(*mocks_->demuxer(), IsSeekable())
-        .WillRepeatedly(Return(kSeekable));
 
     // Configure the demuxer to return the streams.
     for (size_t i = 0; i < streams->size(); ++i) {
@@ -800,24 +794,6 @@ TEST_F(PipelineTest, StartTimeIsNonZero) {
   EXPECT_EQ(kStartTime, pipeline_->GetCurrentTime());
 }
 
-TEST_F(PipelineTest, DemuxerProperties) {
-  CreateAudioStream();
-  CreateVideoStream();
-  MockDemuxerStreamVector streams;
-  streams.push_back(audio_stream());
-  streams.push_back(video_stream());
-
-  InitializeDemuxer(&streams);
-  InitializeAudioDecoder(audio_stream());
-  InitializeAudioRenderer();
-  InitializeVideoDecoder(video_stream());
-  InitializeVideoRenderer();
-  InitializePipeline(PIPELINE_OK);
-
-  EXPECT_EQ(kLocalSource, pipeline_->IsLocalSource());
-  EXPECT_NE(kSeekable, pipeline_->IsStreaming());
-}
-
 class FlexibleCallbackRunner : public base::DelegateSimpleThread::Delegate {
  public:
   FlexibleCallbackRunner(base::TimeDelta delay, PipelineStatus status,