Add polling and audio check to copresence.

components/copresence/mediums/audio/audio_manager_impl.cc

 // Copyright 2014 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 "components/copresence/mediums/audio/audio_manager_impl.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/logging.h"
 #include "base/run_loop.h"
 #include "base/strings/string_util.h"
 #include "components/copresence/mediums/audio/audio_player_impl.h"
 #include "components/copresence/mediums/audio/audio_recorder_impl.h"
 #include "components/copresence/public/copresence_constants.h"
+#include "components/copresence/public/whispernet_client.h"
 #include "content/public/browser/browser_thread.h"
 #include "media/audio/audio_manager.h"
 #include "media/audio/audio_manager_base.h"
 #include "media/base/audio_bus.h"
 
 namespace copresence {
 
 namespace {
 
 // UrlSafe is defined as:
 // '/' represented by a '_' and '+' represented by a '-'
 // TODO(rkc): Move this processing to the whispernet wrapper.
 std::string FromUrlSafe(std::string token) {
   base::ReplaceChars(token, "-", "+", &token);
   base::ReplaceChars(token, "_", "/", &token);
   return token;
 }
 
 const int kSampleExpiryTimeMs = 60 * 60 * 1000;  // 60 minutes.
 const int kMaxSamples = 10000;
 
 }  // namespace
 
 // Public methods.
 
-AudioManagerImpl::AudioManagerImpl() : recorder_(nullptr) {
+AudioManagerImpl::AudioManagerImpl()
+    : whispernet_client_(nullptr), recorder_(nullptr) {
   // TODO(rkc): Move all of these into initializer lists once it is allowed.
   playing_[AUDIBLE] = false;
   playing_[INAUDIBLE] = false;
   recording_[AUDIBLE] = false;
   recording_[INAUDIBLE] = false;
+  heard_own_token_[AUDIBLE] = false;
+  heard_own_token_[INAUDIBLE] = false;
 
   player_[AUDIBLE] = nullptr;
   player_[INAUDIBLE] = nullptr;
 }
 
-void AudioManagerImpl::Initialize(const DecodeSamplesCallback& decode_cb,
-                                  const EncodeTokenCallback& encode_cb) {
+void AudioManagerImpl::Initialize(WhispernetClient* whispernet_client,
+                                  const TokensCallback& tokens_cb) {
   samples_cache_.resize(2);
   samples_cache_[AUDIBLE] = new SamplesMap(
       base::TimeDelta::FromMilliseconds(kSampleExpiryTimeMs), kMaxSamples);
   samples_cache_[INAUDIBLE] = new SamplesMap(
       base::TimeDelta::FromMilliseconds(kSampleExpiryTimeMs), kMaxSamples);
 
-  decode_cb_ = decode_cb;
-  encode_cb_ = encode_cb;
+  whispernet_client_ = whispernet_client;

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

DCHECK this. You can pass in stubs in from the tests now.

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

Done.

+  tokens_cb_ = tokens_cb;
+
+  // These will be unregistered on destruction, so unretained is safe to use.
+  whispernet_client_->RegisterTokensCallback(
+      base::Bind(&AudioManagerImpl::OnTokensFound, base::Unretained(this)));
+  whispernet_client_->RegisterSamplesCallback(
+      base::Bind(&AudioManagerImpl::OnTokenEncoded, base::Unretained(this)));
 
   if (!player_[AUDIBLE])
     player_[AUDIBLE] = new AudioPlayerImpl();
   player_[AUDIBLE]->Initialize();
 
   if (!player_[INAUDIBLE])
     player_[INAUDIBLE] = new AudioPlayerImpl();
   player_[INAUDIBLE]->Initialize();
 
-  decode_cancelable_cb_.Reset(base::Bind(
-      &AudioManagerImpl::DecodeSamplesConnector, base::Unretained(this)));
+  decode_cancelable_cb_.Reset(base::Bind(&WhispernetClient::DecodeSamples,
+                                         base::Unretained(whispernet_client_),
+                                         BOTH));
   if (!recorder_)
     recorder_ = new AudioRecorderImpl();
   recorder_->Initialize(decode_cancelable_cb_.callback());
 }
 
 AudioManagerImpl::~AudioManagerImpl() {
   if (player_[AUDIBLE])
     player_[AUDIBLE]->Finalize();
   if (player_[INAUDIBLE])
     player_[INAUDIBLE]->Finalize();
   if (recorder_)
     recorder_->Finalize();
+
+  if (whispernet_client_) {

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

No if since this will be DCHECK'ed

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

Done but breaks the RPC handler test. Had to fix that.
Done.

+    whispernet_client_->RegisterTokensCallback(TokensCallback());
+    whispernet_client_->RegisterSamplesCallback(SamplesCallback());
+  }
 }
 
 void AudioManagerImpl::StartPlaying(AudioType type) {
   DCHECK(type == AUDIBLE || type == INAUDIBLE);
   playing_[type] = true;
   // If we don't have our token encoded yet, this check will be false, for now.
   // Once our token is encoded, OnTokenEncoded will call UpdateToken, which
   // will call this code again (if we're still supposed to be playing).
-  if (samples_cache_[type]->HasKey(token_[type]) &&
+  if (samples_cache_[type]->HasKey(playing_token_[type]) &&
       !player_[type]->IsPlaying()) {
-    DCHECK(!token_[type].empty());
-    player_[type]->Play(samples_cache_[type]->GetValue(token_[type]));
+    DCHECK(!playing_token_[type].empty());
+    player_[type]->Play(samples_cache_[type]->GetValue(playing_token_[type]));
+    // If we're playing, we always record to hear what we are playing.
+    if (!recorder_->IsRecording())
+      recorder_->Record();
   }
 }
 
 void AudioManagerImpl::StopPlaying(AudioType type) {
   DCHECK(type == AUDIBLE || type == INAUDIBLE);
   playing_[type] = false;
-  if (player_[type]->IsPlaying())
+  if (player_[type]->IsPlaying()) {
     player_[type]->Stop();
+    // If we were only recording to hear our own played tokens, stop.
+    if (recorder_->IsRecording() && !recording_[AUDIBLE] &&
+        !recording_[INAUDIBLE])
+      recorder_->Stop();
+  }
 }
 
 void AudioManagerImpl::StartRecording(AudioType type) {
   DCHECK(type == AUDIBLE || type == INAUDIBLE);
   recording_[type] = true;
   if (!recorder_->IsRecording())
     recorder_->Record();
 }
 
 void AudioManagerImpl::StopRecording(AudioType type) {
   DCHECK(type == AUDIBLE || type == INAUDIBLE);
   recording_[type] = false;
   if (recorder_->IsRecording())
     recorder_->Stop();
 }
 
 void AudioManagerImpl::SetToken(AudioType type,
                                 const std::string& url_unsafe_token) {
   DCHECK(type == AUDIBLE || type == INAUDIBLE);
   std::string token = FromUrlSafe(url_unsafe_token);
   if (!samples_cache_[type]->HasKey(token)) {
     // We're destructed by the destruction chain of

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

This comment is out of date.

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

Done.

     // RpcHandler->DirectiveHandler->AudioDirectiveHandler. The RpcHandler
     // unsets any callbacks that were set on the Whispernet client before it
     // destructs, unsetting this callback too - making unretained safe to use.
-    encode_cb_.Run(
-        token,
-        type,
-        base::Bind(&AudioManagerImpl::OnTokenEncoded, base::Unretained(this)));
+    whispernet_client_->EncodeToken(token, type);
   } else {
     UpdateToken(type, token);
   }
 }
 
 const std::string AudioManagerImpl::GetToken(AudioType type) {
-  return token_[type];
+  return playing_token_[type];
 }
 
 bool AudioManagerImpl::IsRecording(AudioType type) {
   return recording_[type];
 }
 
 bool AudioManagerImpl::IsPlaying(AudioType type) {
   return playing_[type];
 }
 
+bool AudioManagerImpl::IsPlayingTokenHeard(AudioType type) {
+  return heard_own_token_[type];
+}
+
 // Private methods.
 
 void AudioManagerImpl::OnTokenEncoded(
+    AudioType type,
     const std::string& token,
-    AudioType type,
     const scoped_refptr<media::AudioBusRefCounted>& samples) {
   samples_cache_[type]->Add(token, samples);
   UpdateToken(type, token);
 }
 
+void AudioManagerImpl::OnTokensFound(const std::vector<AudioToken>& tokens) {
+  std::vector<AudioToken> tokens_to_report;
+  for (const auto& token : tokens) {
+    AudioType type = token.audible ? AUDIBLE : INAUDIBLE;
+    if (playing_token_[type] == token.token)
+      heard_own_token_[type] = true;
+
+    if (recording_[AUDIBLE] && token.audible) {
+      tokens_to_report.push_back(token);
+    } else if (recording_[INAUDIBLE] && !token.audible) {
+      tokens_to_report.push_back(token);
+    }
+  }
+
+  if (!tokens_cb_.is_null() && !tokens_to_report.empty())

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

Why should tokens_cb_ be null?

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

Done.

+    tokens_cb_.Run(tokens_to_report);
+}
+
 void AudioManagerImpl::UpdateToken(AudioType type, const std::string& token) {
   DCHECK(type == AUDIBLE || type == INAUDIBLE);
-  if (token_[type] == token)
+  if (playing_token_[type] == token)
     return;
 
   // Update token.
-  token_[type] = token;
+  playing_token_[type] = token;
 
   // If we are supposed to be playing this token type at this moment, switch
   // out playback with the new samples.
   if (playing_[type]) {
     if (player_[type]->IsPlaying())
       player_[type]->Stop();
     StartPlaying(type);
   }
 }
 
-void AudioManagerImpl::DecodeSamplesConnector(const std::string& samples) {
-  AudioType decode_type = AUDIO_TYPE_UNKNOWN;
-
-  if (recording_[AUDIBLE] && recording_[INAUDIBLE])
-    decode_type = BOTH;
-  else if (recording_[AUDIBLE])
-    decode_type = AUDIBLE;
-  else if (recording_[INAUDIBLE])
-    decode_type = INAUDIBLE;
-
-  decode_cb_.Run(decode_type, samples);
-}
-
 }  // namespace copresence