Refactor PipelineImpl to use CompositeFilter to manage Filter state transitions.

media/base/mock_filters.cc

Index: media/base/mock_filters.cc
diff --git a/media/base/mock_filters.cc b/media/base/mock_filters.cc
index f2110ffa36da014d3cd2e9b562c3077ab1e5897a..ee46a7fd77dff8b69dada0870400289666166b35 100644
--- a/media/base/mock_filters.cc
+++ b/media/base/mock_filters.cc
@@ -20,4 +20,29 @@ void RunStopFilterCallback(FilterCallback* callback) {
   delete callback;
 }
 
+MockFilter::MockFilter() :
+    requires_message_loop_(false),
+    message_loop_name_("MockFilter")
+{}

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

pedantic style nit: I *think* when we have an empty method we have the { on the
previous line

we use the {} form when it's a one-liner (which you could do here for the dtor,
but not so much the ctor)

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

Done.

+
+MockFilter::~MockFilter()
+{}
+
+void MockFilter::set_requires_message_loop(bool requires_message_loop) {
+  requires_message_loop_ =  requires_message_loop;
+}
+
+void MockFilter::set_message_loop_name(const char* name) {
+  message_loop_name_ = name;
+}
+
+bool MockFilter::requires_message_loop() const {
+  return requires_message_loop_;
+}
+
+const char* MockFilter::message_loop_name() const {
+  return message_loop_name_.c_str();
+}
+
+
 }  // namespace media