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

media/base/composite_filter.cc

 // Copyright (c) 2011 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/composite_filter.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
@@ skipping 425 matching lines @@
 
 void CompositeFilter::OnStatusCB(const base::Closure& callback,
                                  PipelineStatus status) {
   if (status != PIPELINE_OK)
     SetError(status);
 
   callback.Run();
 }
 
 void CompositeFilter::SetError(PipelineStatus error) {
-  // TODO(acolwell): Temporary hack to handle errors that occur
-  // during filter initialization. In this case we just forward
-  // the error to the host even if it is on the wrong thread. We
-  // have to do this because if we defer the call, we can't be
-  // sure the host will get the error before the "init done" callback
-  // is executed. This will be cleaned up when filter init is refactored.
-  if (state_ == kCreated) {
-    SendErrorToHost(error);
-    return;
-  }
-
   if (message_loop_ != MessageLoop::current()) {
     message_loop_->PostTask(FROM_HERE,
         base::Bind(&CompositeFilter::SetError, this, error));
     return;
   }
 
   DCHECK_EQ(message_loop_, MessageLoop::current());

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

DCHECK_NE(state_, kCreated);

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

Done.

 
   // Drop errors recieved while stopping or stopped.
   // This shields the owner of this object from having
   // to deal with errors it can't do anything about.
   if (state_ == kStopPending || state_ == kStopped)
     return;
 
   status_ = error;
   if (CanForwardError())
     SendErrorToHost(error);
@@ skipping 33 matching lines @@
 
 void CompositeFilter::FilterHostImpl::NotifyEnded() {
   host_->NotifyEnded();
 }
 
 void CompositeFilter::FilterHostImpl::DisableAudioRenderer() {
   host_->DisableAudioRenderer();
 }
 
 }  // namespace media