Pass delay and timestamp to AudioSourceCallback::OnMoreData.

content/browser/media/capture/web_contents_audio_muter.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 "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"
 #include "content/public/browser/web_contents.h"
 #include "media/audio/audio_io.h"
 #include "media/audio/audio_manager.h"
 #include "media/audio/fake_audio_worker.h"
 #include "media/base/bind_to_current_loop.h"
 
 namespace content {
@@ skipping 23 matching lines @@
   }
   void Stop() override { worker_.Stop(); }
   void SetVolume(double volume) override {}
   void GetVolume(double* volume) override { *volume = 0; }
   void Close() override { delete this; }
 
  private:
   ~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.
   media::FakeAudioWorker worker_;
   std::unique_ptr<media::AudioBus> audio_bus_;
 
   DISALLOW_COPY_AND_ASSIGN(AudioDiscarder);
 };
 
 }  // namespace
@@ skipping 87 matching lines @@
     return;
   is_muting_ = false;
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&AudioMirroringManager::StopMirroring,
                  base::Unretained(AudioMirroringManager::GetInstance()),
                  base::RetainedRef(destination_)));
 }
 
 }  // namespace content