cros: Enable new cras audio handler by default

chrome/browser/ui/ash/volume_controller_browsertest_chromeos.cc

Index: chrome/browser/ui/ash/volume_controller_browsertest_chromeos.cc
diff --git a/chrome/browser/ui/ash/volume_controller_browsertest_chromeos.cc b/chrome/browser/ui/ash/volume_controller_browsertest_chromeos.cc
index bd6fdfdbaba0f5040265038f220cea9922d0aa36..c4e31b1a094a49a00aeaba197fc200b8a7ba07f5 100644
--- a/chrome/browser/ui/ash/volume_controller_browsertest_chromeos.cc
+++ b/chrome/browser/ui/ash/volume_controller_browsertest_chromeos.cc
@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "ash/ash_switches.h"
+#include "base/command_line.h"
+#include "base/memory/scoped_ptr.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/audio/audio_handler.h"
 #include "chrome/browser/chromeos/audio/audio_mixer.h"
@@ -82,6 +85,8 @@ class VolumeControllerTest : public InProcessBrowserTest {
   VolumeControllerTest() {}
 
   virtual void SetUpOnMainThread() OVERRIDE {
+    volume_controller_.reset(new VolumeController());
+
     // First we should shutdown the default audio handler.
     chromeos::AudioHandler::Shutdown();
     audio_mixer_ = new MockAudioMixer;
@@ -94,31 +99,35 @@ class VolumeControllerTest : public InProcessBrowserTest {
     audio_mixer_ = NULL;

[James Cook, 2013/04/30 23:31:21]

Perhaps not necessary, but I would do volume_controller_.reset() here.

[rkc, 2013/04/30 23:36:37]

It's a scoped_ptr, the next .reset will cleanup the previous one. I made it a
scoped_ptr precisely so I wouldn't need to clean it up :)

   }
 
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    command_line->AppendSwitch(ash::switches::kAshDisableNewAudioHandler);

[James Cook, 2013/04/30 23:31:21]

Maybe a todo and a bug to track fixing this test?

[rkc, 2013/04/30 23:36:37]

Added the TODO. Once we remove the old Audio Handler code, this code will be
directly affected - I don't believe it will be possible to check in that code
without fixing this test, hence skipping the creating a bug for it for the
moment.

+  }
+
  protected:
   void SetVolumePercent(double percent) {
-    volume_controller_.SetVolumePercent(percent);
+    volume_controller_->SetVolumePercent(percent);
   }
 
   void VolumeMute() {
-    volume_controller_.HandleVolumeMute(ui::Accelerator());
+    volume_controller_->HandleVolumeMute(ui::Accelerator());
   }
 
   void VolumeUnmute() {
-    volume_controller_.SetAudioMuted(false);
+    volume_controller_->SetAudioMuted(false);
   }
 
   void VolumeUp() {
-    volume_controller_.HandleVolumeUp(ui::Accelerator());
+    volume_controller_->HandleVolumeUp(ui::Accelerator());
   }
 
   void VolumeDown() {
-    volume_controller_.HandleVolumeDown(ui::Accelerator());
+    volume_controller_->HandleVolumeDown(ui::Accelerator());
   }
 
   MockAudioMixer* audio_mixer() { return audio_mixer_; }
 
  private:
-  VolumeController volume_controller_;
+  scoped_ptr<VolumeController> volume_controller_;
   MockAudioMixer* audio_mixer_;
 
   DISALLOW_COPY_AND_ASSIGN(VolumeControllerTest);