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

remoting/client/audio_decode_scheduler_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "remoting/client/audio_decode_scheduler.h"
+
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/threading/thread.h"
 #include "remoting/base/auto_thread.h"
 #include "remoting/base/auto_thread_task_runner.h"
-#include "remoting/client/audio_decode_scheduler.h"
 #include "remoting/client/fake_audio_consumer.h"
+#include "remoting/proto/audio.pb.h"
 #include "remoting/protocol/session_config.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 const int kAudioSampleBytes = 4;
 const uint8_t kDummyAudioData = 0x8B;
 
 }  // namespace
 
@@ skipping 23 matching lines @@
 
 void AudioDecodeSchedulerTest::TearDown() {
   // Release the task runners, so that the test can quit.
   audio_decode_task_runner_ = nullptr;
   main_task_runner_ = nullptr;
 
   // Run the MessageLoop until everything has torn down.
   run_loop_.Run();
 }
 
-// TODO(nicholss) could share the following in a common class for use
-// other places.
+// TODO(nicholss): Could share the following in a common class for use
+// in other places.
 std::unique_ptr<AudioPacket> CreatePacketWithSamplingRate_(
     AudioPacket::SamplingRate rate,
     int samples) {
   std::unique_ptr<AudioPacket> packet(new AudioPacket());
   packet->set_encoding(AudioPacket::ENCODING_RAW);
   packet->set_sampling_rate(rate);
   packet->set_bytes_per_sample(AudioPacket::BYTES_PER_SAMPLE_2);
   packet->set_channels(AudioPacket::CHANNELS_STEREO);
 
   // The data must be a multiple of 4 bytes (channels x bytes_per_sample).
@@ skipping 20 matching lines @@
       new AudioDecodeScheduler(main_task_runner_, audio_decode_task_runner_,
                                audio_consumer->GetWeakPtr()));
 
   audio_scheduler->Initialize(*session_config_);
 
   audio_scheduler->ProcessAudioPacket(CreatePacket44100Hz_(1000),
                                       base::Bind(&base::DoNothing));
 
   audio_scheduler.reset();
   audio_consumer.reset();
+  // TODO(nicholss): This test does not really test anything. Add a way to get
+  // a count of the calls to AddAudioPacket.
 }
 
 }  // namespace remoting