Change audio renderer to communicate with host using low latency codepath.

content/renderer/media/audio_renderer_impl_unittest.cc

Index: content/renderer/media/audio_renderer_impl_unittest.cc
===================================================================
--- content/renderer/media/audio_renderer_impl_unittest.cc	(revision 90349)
+++ content/renderer/media/audio_renderer_impl_unittest.cc	(working copy)
@@ -13,11 +13,33 @@
 
 using ::testing::Return;
 
+// Class we would be tesing. The only difference between it and "real" one
+// is that test class does not open sockets and launch audio thread.
+class TestAudioRendererImpl : public AudioRendererImpl {
+ public:
+  explicit TestAudioRendererImpl(AudioMessageFilter* filter)
+      : AudioRendererImpl(filter) {
+  }
+ private:
+  virtual void CreateSocket(base::SyncSocket::Handle socket_handle) {

[scherkus (not reviewing), 2011/06/24 21:09:26]

nit: for empty one-liner functions we typically close the braces {} assuming the
entire thing can fit on a single line

[enal1, 2011/06/24 22:01:09]

Done.

+  }
+  virtual void CreateAudioThread() {
+  }
+};
+
 class AudioRendererImplTest : public ::testing::Test {
  public:
   static const int kRouteId = 0;
   static const int kSize = 1024;
 
+  void CallOnCreated(base::SharedMemoryHandle handle) {
+    if (renderer_->latency_type() == AudioRendererImpl::kHighLatency) {
+      renderer_->OnCreated(handle, kSize);
+    } else {
+      renderer_->OnLowLatencyCreated(handle, 0, kSize);
+    }
+  }
+
   AudioRendererImplTest() {
     message_loop_.reset(new MessageLoop(MessageLoop::TYPE_IO));
 
@@ -38,10 +60,13 @@
                                                         44100)));
 
     // Create and initialize audio renderer.
-    renderer_ = new AudioRendererImpl(filter_);
+    renderer_ = new TestAudioRendererImpl(filter_);
     renderer_->set_host(&host_);
     renderer_->Initialize(decoder_, media::NewExpectedCallback());
 
+    // Set low latency mode, as it soon would be on by default.
+    renderer_->set_latency_type(AudioRendererImpl::kLowLatency);
+
     // Run pending tasks and simulate responding with a created audio stream.
     message_loop_->RunAllPending();
 
@@ -50,8 +75,7 @@
     base::SharedMemoryHandle duplicated_handle;
     EXPECT_TRUE(shared_mem_.ShareToProcess(base::GetCurrentProcessHandle(),
                                            &duplicated_handle));
-
-    renderer_->OnCreated(duplicated_handle, kSize);
+    CallOnCreated(duplicated_handle);
   }
 
   virtual ~AudioRendererImplTest() {
@@ -96,11 +120,13 @@
 
   // Run AudioMessageFilter::Delegate methods, which can be executed after being
   // stopped.  AudioRendererImpl shouldn't create any messages.
-  renderer_->OnRequestPacket(AudioBuffersState(kSize, 0));
+  if (renderer_->latency_type() == AudioRendererImpl::kHighLatency) {

[scherkus (not reviewing), 2011/06/24 21:09:26]

if every test has its latency set as kLowLatency then will this ever get
executed?

[enal1, 2011/06/24 22:01:09]

I prefer code that can be executed in both high and low latency modes. E.g. I
run those tests in high latency mode to verify I did not break it...

+    renderer_->OnRequestPacket(AudioBuffersState(kSize, 0));
+  }
   renderer_->OnStateChanged(kAudioStreamError);
   renderer_->OnStateChanged(kAudioStreamPlaying);
   renderer_->OnStateChanged(kAudioStreamPaused);
-  renderer_->OnCreated(shared_mem_.handle(), kSize);
+  CallOnCreated(shared_mem_.handle());
   renderer_->OnVolume(0.5);
 
   // It's possible that the upstream decoder replies right after being stopped.