Update AudioRenderer, VideoRenderer, and AudioDecoder Initialize() methods to use PipelineStatusCB.

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 ad3bc38cb992b0834d68a105020e14a7e3ba721d..484fe8dc459b3e60b14a58359d9fffee594ba46f 100644
--- a/media/base/mock_filters.cc
+++ b/media/base/mock_filters.cc
@@ -171,10 +171,9 @@ void RunPipelineStatusCB(PipelineStatus status, const PipelineStatusCB& cb) {
   cb.Run(status);
 }
 
-void RunFilterCallback3(::testing::Unused, const base::Closure& callback,
-                        ::testing::Unused) {
-  callback.Run();
-
+void RunPipelineStatusCB3(::testing::Unused, const PipelineStatusCB& callback,
+                          ::testing::Unused) {
+  callback.Run(PIPELINE_OK);

[Ami GONE FROM CHROMIUM, 2012/02/01 18:39:11]

The fact that this method doesn't take a |status| param (and always fires
PIPELINE_OK) is suspicious (might simply indicate a lack of test coverage).

[acolwell GONE FROM CHROMIUM, 2012/02/01 19:59:46]

Agreed. Created a bug for adding initialization failure tests.
http://crbug.com/112337

 }
 
 void RunStopFilterCallback(const base::Closure& callback) {