Add polling and audio check to copresence.

components/copresence/handlers/audio/audio_directive_handler_unittest.cc

Index: components/copresence/handlers/audio/audio_directive_handler_unittest.cc
diff --git a/components/copresence/handlers/audio/audio_directive_handler_unittest.cc b/components/copresence/handlers/audio/audio_directive_handler_unittest.cc
index 0400662149ec58ca854406e17abf6fe5ac2caadc..ce45e690a76a157f7439bd6d2aeff7a7e370b625 100644
--- a/components/copresence/handlers/audio/audio_directive_handler_unittest.cc
+++ b/components/copresence/handlers/audio/audio_directive_handler_unittest.cc
@@ -16,26 +16,14 @@
 
 namespace copresence {
 
-namespace {
-
-// Callback stubs to pass into the directive handler.
-void DecodeSamples(AudioType, const std::string&) {
-}
-void EncodeToken(const std::string&,
-                 AudioType,
-                 const AudioManager::SamplesCallback&) {
-}
-
-}  // namespace
-
 class AudioManagerStub final : public AudioManager {
  public:
   AudioManagerStub() {}
   ~AudioManagerStub() override {}
 
   // AudioManager overrides:
-  void Initialize(const DecodeSamplesCallback& decode_cb,
-                  const EncodeTokenCallback& encode_cb) override {}
+  void Initialize(WhispernetClient* whispernet_client,
+                  const TokensCallback& tokens_cb) override {}
   void StartPlaying(AudioType type) override { playing_[type] = true; }
   void StopPlaying(AudioType type) override { playing_[type] = false; }
   void StartRecording(AudioType type) override { recording_[type] = true; }
@@ -44,6 +32,7 @@ class AudioManagerStub final : public AudioManager {
   const std::string GetToken(AudioType type) override { return std::string(); }
   bool IsRecording(AudioType type) override { return recording_[type]; }
   bool IsPlaying(AudioType type) override { return playing_[type]; }
+  bool IsPlayingTokenHeard(AudioType type) override { return false; }
 
  private:
   // Indexed using enum AudioType.
@@ -64,8 +53,7 @@ class AudioDirectiveHandlerTest : public testing::Test {
         make_scoped_ptr<AudioManager>(manager_ptr_),
         make_scoped_ptr<base::Timer>(timer_ptr_),
         make_scoped_refptr(new TickClockRefCounted(clock_ptr_))));
-    directive_handler_->Initialize(base::Bind(&DecodeSamples),
-                                   base::Bind(&EncodeToken));
+    directive_handler_->Initialize(NULL, TokensCallback());

[Charlie, 2014/11/06 17:28:22]

Pass a StubWhispernetClient here instead (owned by a local scoped_ptr). And
DCHECK whispernet in the AudioManager.

[rkc, 2014/11/06 19:58:24]

Why do I need to pass in a Stub to DCHECK in the AudioManager?

[Charlie, 2014/11/06 21:53:14]

In the real implementation, WhispernetClient shouldn't be null, and we want an
error when it is. Tests shouldn't interfere with prod behavior.

[rkc, 2014/11/06 21:58:33]

We will never get to the point that Whispernet will be used. That is the whole
point of having a stub. If at any point this behavior changes, this *test* will
crash, so we should be fine.

   }
   ~AudioDirectiveHandlerTest() override {}