Refactor PipelineImpl to use CompositeFilter to manage Filter state transitions.

media/base/mock_filters.h

Index: media/base/mock_filters.h
diff --git a/media/base/mock_filters.h b/media/base/mock_filters.h
index 3b1deb31e9ddbd6599110f90ecb3861a5e574981..132d49779bfac773b7b3ea018ab7ce1b9730e2ab 100644
--- a/media/base/mock_filters.h
+++ b/media/base/mock_filters.h
@@ -90,6 +90,35 @@ class MockFilterCallback {
   DISALLOW_COPY_AND_ASSIGN(MockFilterCallback);
 };
 
+class MockFilter : public Filter {
+ public:
+  MockFilter();
+
+  void set_requires_message_loop(bool requires_message_loop);

[scherkus (not reviewing), 2010/12/15 16:44:02]

looks like these are always called after creation.. perhaps make ctor arguments
instead?

[acolwell GONE FROM CHROMIUM, 2010/12/15 18:20:11]

Done.

+  void set_message_loop_name(const char* name);
+
+  // Filter implementation.
+  virtual bool requires_message_loop() const;
+  virtual const char* message_loop_name() const;
+
+  MOCK_METHOD1(Play, void(FilterCallback* callback));
+  MOCK_METHOD1(Pause, void(FilterCallback* callback));
+  MOCK_METHOD1(Flush, void(FilterCallback* callback));
+  MOCK_METHOD1(Stop, void(FilterCallback* callback));
+  MOCK_METHOD1(SetPlaybackRate, void(float playback_rate));
+  MOCK_METHOD2(Seek, void(base::TimeDelta time, FilterCallback* callback));
+  MOCK_METHOD0(OnAudioRendererDisabled, void());
+
+ protected:
+  virtual ~MockFilter();
+
+ private:
+  bool requires_message_loop_;
+  std::string message_loop_name_;
+
+  DISALLOW_COPY_AND_ASSIGN(MockFilter);
+};
+
 class MockDataSource : public DataSource {
  public:
   MockDataSource() {}