Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Unified Diff: chromecast/media/audio/cast_audio_output_stream_unittest.cc

Issue 2101303004: Pass delay and timestamp to AudioSourceCallback::OnMoreData. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Fix Mac CQ errors. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromecast/media/audio/cast_audio_output_stream_unittest.cc
diff --git a/chromecast/media/audio/cast_audio_output_stream_unittest.cc b/chromecast/media/audio/cast_audio_output_stream_unittest.cc
index 1f3f7b3d9bc99f7355181182a93576a26a1202fb..ec23f03a19691539c96fb1433d7035af52061ebc 100644
--- a/chromecast/media/audio/cast_audio_output_stream_unittest.cc
+++ b/chromecast/media/audio/cast_audio_output_stream_unittest.cc
@@ -4,6 +4,9 @@
#include "chromecast/media/audio/cast_audio_output_stream.h"
+#include <memory>
+#include <utility>
+
#include "base/memory/ptr_util.h"
#include "base/run_loop.h"
#include "chromecast/base/metrics/cast_metrics_test_helper.h"
@@ -152,11 +155,12 @@ class FakeAudioSourceCallback
bool error() const { return error_; }
// ::media::AudioOutputStream::AudioSourceCallback overrides.
- int OnMoreData(::media::AudioBus* audio_bus,
- uint32_t total_bytes_delay,
- uint32_t frames_skipped) override {
- audio_bus->Zero();
- return audio_bus->frames();
+ int OnMoreData(base::TimeDelta /* delay */,
+ base::TimeTicks /* delay_timestamp */,
+ int /* prior_frames_skipped */,
+ ::media::AudioBus* dest) override {
+ dest->Zero();
+ return dest->frames();
}
void OnError(::media::AudioOutputStream* stream) override { error_ = true; }
@@ -166,7 +170,8 @@ class FakeAudioSourceCallback
class FakeAudioManager : public CastAudioManager {
public:
- FakeAudioManager(scoped_refptr<base::SingleThreadTaskRunner> task_runner)
+ explicit FakeAudioManager(
+ scoped_refptr<base::SingleThreadTaskRunner> task_runner)
: CastAudioManager(task_runner, task_runner, nullptr, nullptr, nullptr),
media_pipeline_backend_(nullptr) {}
~FakeAudioManager() override {}

Powered by Google App Engine
This is Rietveld 408576698