Refactor PipelineImpl to use CompositeFilter to manage Filter state transitions.

media/base/mock_filters.cc

 // Copyright (c) 2010 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/mock_filters.h"
 
 namespace media {
 
 void RunFilterCallback(::testing::Unused, FilterCallback* callback) {
   callback->Run();
   delete callback;
 }
 
 void DestroyFilterCallback(::testing::Unused, FilterCallback* callback) {
   delete callback;
 }
 
 void RunStopFilterCallback(FilterCallback* callback) {
   callback->Run();
   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