Remove completion callbacks from AudioRenderer::Play/Pause().

media/filters/audio_renderer_impl.cc

 // Copyright (c) 2012 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/filters/audio_renderer_impl.h"
 
 #include <math.h>
 
 #include <algorithm>
 
@@ skipping 55 matching lines @@
   audio_buffer_stream_.set_config_change_observer(base::Bind(
       &AudioRendererImpl::OnConfigChange, weak_factory_.GetWeakPtr()));
 }
 
 AudioRendererImpl::~AudioRendererImpl() {
   // Stop() should have been called and |algorithm_| should have been destroyed.
   DCHECK(state_ == kUninitialized || state_ == kStopped);
   DCHECK(!algorithm_.get());
 }
 
-void AudioRendererImpl::Play(const base::Closure& callback) {
+void AudioRendererImpl::Play() {
   DCHECK(task_runner_->BelongsToCurrentThread());
 
   base::AutoLock auto_lock(lock_);
   DCHECK_EQ(state_, kPaused);
   ChangeState_Locked(kPlaying);
-  callback.Run();
   earliest_end_time_ = now_cb_.Run();
 
   if (algorithm_->playback_rate() != 0)
     DoPlay_Locked();
   else
     DCHECK(!sink_playing_);
 }
 
 void AudioRendererImpl::DoPlay_Locked() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   lock_.AssertAcquired();
   earliest_end_time_ = now_cb_.Run();
 
   if ((state_ == kPlaying || state_ == kRebuffering || state_ == kUnderflow) &&
       !sink_playing_) {
     {
       base::AutoUnlock auto_unlock(lock_);
       sink_->Play();
     }
 
     sink_playing_ = true;
   }
 }
 
-void AudioRendererImpl::Pause(const base::Closure& callback) {
+void AudioRendererImpl::Pause() {
   DCHECK(task_runner_->BelongsToCurrentThread());
 
   base::AutoLock auto_lock(lock_);
   DCHECK(state_ == kPlaying || state_ == kUnderflow ||
          state_ == kRebuffering) << "state_ == " << state_;
   ChangeState_Locked(kPaused);
 
   DoPause_Locked();
-
-  callback.Run();
 }
 
 void AudioRendererImpl::DoPause_Locked() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   lock_.AssertAcquired();
 
   if (sink_playing_) {
     {
       base::AutoUnlock auto_unlock(lock_);
       sink_->Pause();
@@ skipping 619 matching lines @@
   DCHECK(expecting_config_changes_);
   buffer_converter_->ResetTimestampState();
   // Drain flushed buffers from the converter so the AudioSplicer receives all
   // data ahead of any OnNewSpliceBuffer() calls.  Since discontinuities should
   // only appear after config changes, AddInput() should never fail here.
   while (buffer_converter_->HasNextBuffer())
     CHECK(splicer_->AddInput(buffer_converter_->GetNextBuffer()));
 }
 
 }  // namespace media