Fix AudioCapturer implementation to read from audio pipe even when there are no active clients.

remoting/host/remoting_me2me_host.cc

Index: remoting/host/remoting_me2me_host.cc
diff --git a/remoting/host/remoting_me2me_host.cc b/remoting/host/remoting_me2me_host.cc
index bf23e290d7b0695e9b5a5171d788550043b9a38f..351af8d7cacee3843db7d795c6cd0c66500d42a3 100644
--- a/remoting/host/remoting_me2me_host.cc
+++ b/remoting/host/remoting_me2me_host.cc
@@ -444,6 +444,16 @@ void HostProcess::StartHostProcess() {
     return;
   }
 
+#if defined(OS_LINUX)
+  // TODO(sergeyu): Pass configuration parameters to the Linux-specific version
+  // of DesktopEnvironmentFactory when we have it.
+  remoting::VideoFrameCapturer::EnableXDamage(true);
+  remoting::AudioCapturerLinuxCore::Initialize(
+      context_->audio_task_runner(),
+      CommandLine::ForCurrentProcess()->GetSwitchValuePath(
+          kAudioPipeSwitchName));

[Wez, 2012/10/30 04:20:11]

nit: I think this call has become sufficiently complex to justify a comment
explaining that we're passing the PulseAudio source path, and the TaskRunner to
use to monitor it.

[Sergey Ulanov, 2012/10/30 23:59:48]

Done.

+#endif  // defined(OS_LINUX)
+
   // Create a desktop environment factory appropriate to the build type &
   // platform.
 #if defined(OS_WIN)
@@ -892,14 +902,6 @@ int main(int argc, char** argv) {
           new remoting::AutoThreadTaskRunner(message_loop.message_loop_proxy(),
                                              quit_message_loop)));
 
-#if defined(OS_LINUX)
-  // TODO(sergeyu): Pass configuration parameters to the Linux-specific version
-  // of DesktopEnvironmentFactory when we have it.
-  remoting::VideoFrameCapturer::EnableXDamage(true);
-  remoting::AudioCapturerLinux::SetPipeName(CommandLine::ForCurrentProcess()->
-      GetSwitchValuePath(kAudioPipeSwitchName));
-#endif  // defined(OS_LINUX)
-
   if (!context->Start())
     return remoting::kInitializationFailed;