Allow AudioRendererMixerInputs to be restarted after stopped.

media/base/audio_renderer_mixer_input_unittest.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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "media/base/audio_renderer_mixer.h"
 #include "media/base/audio_renderer_mixer_input.h"
 #include "media/base/fake_audio_render_callback.h"
 #include "media/base/mock_audio_renderer_sink.h"
@@ skipping 10 matching lines @@
 class AudioRendererMixerInputTest : public testing::Test {
  public:
   AudioRendererMixerInputTest() {
     audio_parameters_ = AudioParameters(
         AudioParameters::AUDIO_PCM_LINEAR, kChannelLayout, kSampleRate,
         kBitsPerChannel, kBufferSize);
 
     CreateMixerInput();
     fake_callback_.reset(new FakeAudioRenderCallback(0));
     mixer_input_->Initialize(audio_parameters_, fake_callback_.get());
-    EXPECT_CALL(*this, RemoveMixer(testing::_));
     audio_bus_ = AudioBus::Create(audio_parameters_);
   }
 
   void CreateMixerInput() {
     mixer_input_ = new AudioRendererMixerInput(
         base::Bind(
             &AudioRendererMixerInputTest::GetMixer, base::Unretained(this)),
         base::Bind(
             &AudioRendererMixerInputTest::RemoveMixer, base::Unretained(this)));
   }
 
   AudioRendererMixer* GetMixer(const AudioParameters& params) {
     if (!mixer_) {
       scoped_refptr<MockAudioRendererSink> sink = new MockAudioRendererSink();
       EXPECT_CALL(*sink.get(), Start());
       EXPECT_CALL(*sink.get(), Stop());
 
       mixer_.reset(new AudioRendererMixer(
           audio_parameters_, audio_parameters_, sink));
     }
+    EXPECT_CALL(*this, RemoveMixer(testing::_));
     return mixer_.get();
   }
 
   double ProvideInput() {
     return mixer_input_->ProvideInput(audio_bus_.get(), base::TimeDelta());
   }
 
   MOCK_METHOD1(RemoveMixer, void(const AudioParameters&));
 
  protected:
@@ skipping 40 matching lines @@
 // Test that Stop() can be called before Initialize() and Start().
 TEST_F(AudioRendererMixerInputTest, StopBeforeInitializeOrStart) {
   // |mixer_input_| was initialized during construction.
   mixer_input_->Stop();
 
   // Verify Stop() works without Initialize() or Start().
   CreateMixerInput();
   mixer_input_->Stop();
 }
 
+// Test that Start() can be called after Stop().
+// TODO(dalecurtis): We shouldn't allow this.  See http://crbug.com/151051
+TEST_F(AudioRendererMixerInputTest, StartAfterStop) {
+  mixer_input_->Stop();
+  mixer_input_->Start();
+  mixer_input_->Stop();
+}
+
 }  // namespace media