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

remoting/client/BUILD.gn

Index: remoting/client/BUILD.gn
diff --git a/remoting/client/BUILD.gn b/remoting/client/BUILD.gn
index b8f155888d0cf2df9486f5598dc8eade1bac5261..bbd8e6fefda2a7656192e2c117dbc339cdbd74b8 100644
--- a/remoting/client/BUILD.gn
+++ b/remoting/client/BUILD.gn
@@ -41,6 +41,20 @@ source_set("client") {
   }
 }
 
+source_set("test_support") {

[Lambros, 2016/06/17 23:51:09]

Why does this need to be a separate target? Couldn't you add these files to the
"unit_tests" sources list?

[nicholss, 2016/06/20 17:47:51]

Just following the patterns seen elsewhere. The unit_tests target seems to only
have unit test files so I thought it would be wrong to add a non testing file to
the list. Elsewhere in the codebase this is how it is done from what I saw.

+  testonly = true
+
+  sources = [
+    "fake_audio_consumer.cc",
+    "fake_audio_consumer.h",
+  ]
+
+  public_deps = [
+    "//testing/gmock",
+    "//third_party/protobuf:protobuf_lite",
+  ]
+}
+
 source_set("unit_tests") {
   testonly = true
 
@@ -48,6 +62,7 @@ source_set("unit_tests") {
   # be compiled on all platforms.
   set_sources_assignment_filter([])
   sources = [
+    "audio_decode_scheduler_unittest.cc",
     "audio_player_unittest.cc",
     "chromoting_client_runtime_unittest.cc",
     "client_status_logger_unittest.cc",
@@ -70,6 +85,7 @@ source_set("unit_tests") {
 
   deps = [
     ":client",
+    ":test_support",
     "//remoting/proto",
     "//testing/gmock",
     "//testing/gtest",