Adding an interface to allow extention of the audio player for CRD and iOS.

remoting/client/jni/chromoting_jni_instance.cc

Index: remoting/client/jni/chromoting_jni_instance.cc
diff --git a/remoting/client/jni/chromoting_jni_instance.cc b/remoting/client/jni/chromoting_jni_instance.cc
index 6e96c3b6d3ec515857215ae2254e01ef38908cb3..cfa8b97b77a203f1e59fd5f715132beda9672e30 100644
--- a/remoting/client/jni/chromoting_jni_instance.cc
+++ b/remoting/client/jni/chromoting_jni_instance.cc
@@ -46,7 +46,7 @@ const bool kXmppUseTls = true;
 // Interval at which to log performance statistics, if enabled.
 const int kPerfStatsIntervalMs = 60000;
 
-}
+}  // namespace
 
 ChromotingJniInstance::ChromotingJniInstance(
     ChromotingJniRuntime* jni_runtime,
@@ -400,9 +400,13 @@ void ChromotingJniInstance::ConnectToHostOnNetworkThread() {
   video_renderer_.reset(new SoftwareVideoRenderer(
       client_context_->decode_task_runner(), view_.get(), perf_tracker_.get()));
 
+  if (!audio_player_) {

[Lambros, 2016/06/09 19:01:05]

Can remove the 'if', for consistency with the lines above.

audio_player_ should be null here, though that isn't guaranteed until
crbug.com/617471 is fixed.

+    audio_player_.reset(new AudioPlayerAndroid());
+  }
+
   client_.reset(
       new ChromotingClient(client_context_.get(), this, video_renderer_.get(),
-                           base::WrapUnique(new AudioPlayerAndroid())));
+                           audio_player_.get()));
 
   signaling_.reset(
       new XmppSignalStrategy(net::ClientSocketFactory::GetDefaultFactory(),
@@ -503,6 +507,7 @@ void ChromotingJniInstance::LogPerfStats() {
 void ChromotingJniInstance::ReleaseResources() {
   // |client_| must be torn down before |signaling_|.
   client_.reset();
+  audio_player_.reset();
   video_renderer_.reset();
   view_.reset();
   signaling_.reset();