Small refactor to media architecture in order to allow end-to-end tests.

media/audio/test_audio_input_controller_factory.cc

Index: media/audio/test_audio_input_controller_factory.cc
diff --git a/media/audio/test_audio_input_controller_factory.cc b/media/audio/test_audio_input_controller_factory.cc
index c2f5ef8caf82ac02796ddad75af9fa1f9e67d43f..64bfb9f060d6f00daca07a29e38cc0e7a1a6aae3 100644
--- a/media/audio/test_audio_input_controller_factory.cc
+++ b/media/audio/test_audio_input_controller_factory.cc
@@ -10,25 +10,39 @@ namespace media {
 TestAudioInputController::TestAudioInputController(
     TestAudioInputControllerFactory* factory,
     AudioManager* audio_manager,
+    const AudioParameters& audio_parameters,
     EventHandler* event_handler,
     SyncWriter* sync_writer)
     : AudioInputController(event_handler, sync_writer),
+      audio_parameters_(audio_parameters),
       factory_(factory),
       event_handler_(event_handler) {
   message_loop_ = audio_manager->GetMessageLoop();
 }
 
-void TestAudioInputController::Close(const base::Closure& closed_task) {
-  message_loop_->PostTask(FROM_HERE, closed_task);
-}
-
 TestAudioInputController::~TestAudioInputController() {
   // Inform the factory so that it allows creating new instances in future.
   factory_->OnTestAudioInputControllerDestroyed(this);
 }
 
+void TestAudioInputController::Record() {
+  if (factory_->delegate_)
+    factory_->delegate_->TestAudioControllerOpened(this);
+}
+
+void TestAudioInputController::Close(const base::Closure& closed_task) {
+  message_loop_->PostTask(FROM_HERE, closed_task);
+  if (factory_->delegate_)
+    factory_->delegate_->TestAudioControllerClosed(this);
+}
+
 TestAudioInputControllerFactory::TestAudioInputControllerFactory()
-    : controller_(NULL) {
+    : controller_(NULL),
+      delegate_(NULL) {
+}
+
+TestAudioInputControllerFactory::~TestAudioInputControllerFactory() {
+  DCHECK(!controller_);
 }
 
 AudioInputController* TestAudioInputControllerFactory::Create(
@@ -36,11 +50,16 @@ AudioInputController* TestAudioInputControllerFactory::Create(
     AudioInputController::EventHandler* event_handler,
     AudioParameters params) {
   DCHECK(!controller_);  // Only one test instance managed at a time.
-  controller_ = new TestAudioInputController(this, audio_manager,
+  controller_ = new TestAudioInputController(this, audio_manager, params,
       event_handler, NULL);
   return controller_;
 }
 
+void TestAudioInputControllerFactory::SetDelegateForTests(

[Ami GONE FROM CHROMIUM, 2012/08/06 23:12:09]

This is a dead method (nobody calls it).  Are there plans to use it sometime
soon or should it be deleted?
(if the latter, should more of this CL be reverted?)

+    TestAudioInputControllerDelegate* delegate) {
+  delegate_ = delegate;
+}
+
 void TestAudioInputControllerFactory::OnTestAudioInputControllerDestroyed(
     TestAudioInputController* controller) {
   DCHECK_EQ(controller_, controller);