Pass delay and timestamp to AudioSourceCallback::OnMoreData.

content/browser/media/capture/web_contents_audio_muter.cc

Index: content/browser/media/capture/web_contents_audio_muter.cc
diff --git a/content/browser/media/capture/web_contents_audio_muter.cc b/content/browser/media/capture/web_contents_audio_muter.cc
index 63df637666d355e7e6b35ebc87a965ed43ea0272..bf6bacbb9b52d0a6741f38489d7ba898304e9978 100644
--- a/content/browser/media/capture/web_contents_audio_muter.cc
+++ b/content/browser/media/capture/web_contents_audio_muter.cc
@@ -4,9 +4,12 @@
 
 #include "content/browser/media/capture/web_contents_audio_muter.h"
 
+#include <set>
+
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/macros.h"
+#include "base/time/time.h"
 #include "content/browser/media/capture/audio_mirroring_manager.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_frame_host.h"
@@ -50,7 +53,7 @@ class AudioDiscarder : public media::AudioOutputStream {
   ~AudioDiscarder() override {}
 
   void FetchAudioData(AudioSourceCallback* callback) {
-    callback->OnMoreData(audio_bus_.get(), 0, 0);
+    callback->OnMoreData(base::TimeTicks(), 0, audio_bus_.get());

[chcunningham, 2016/07/29 01:21:08]

Should the be base::TimeTicks::Now()?

I'm not familiar with this class, but with it historically reporting a delay of
0, I think Now() is the right value to use for target_playout_time.

[chcunningham, 2016/07/29 01:24:12]

Sorry, I didn't say "why" - when you look at how you're computing the
target_playout_time in the various AudioOutputStreams, its generally
Ticks::Now() + delay. So in cases where you have 0 delay, its logical for the
target time to simply be Now()

[jameswest, 2016/08/26 02:08:47]

Done.

   }
 
   // Calls FetchAudioData() at regular intervals and discards the data.