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

Unified Diff: media/base/audio_renderer_mixer_unittest.cc

Issue 2067863003: Mixing audio with different latency requirements (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: android test fix Created 4 years, 6 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
« no previous file with comments | « media/base/audio_renderer_mixer_pool.h ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/audio_renderer_mixer_unittest.cc
diff --git a/media/base/audio_renderer_mixer_unittest.cc b/media/base/audio_renderer_mixer_unittest.cc
index 7af2b829b068cc298b6e2b830887f900d4469e97..46ec57a18686b23bab537fa23c8ea6e8aa82114d 100644
--- a/media/base/audio_renderer_mixer_unittest.cc
+++ b/media/base/audio_renderer_mixer_unittest.cc
@@ -25,6 +25,10 @@
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
+namespace {
+void LogUma(int value) {}
+}
+
namespace media {
// Parameters which control the many input case tests.
@@ -74,7 +78,8 @@ class AudioRendererMixerTest
EXPECT_CALL(*sink_.get(), Start());
EXPECT_CALL(*sink_.get(), Stop());
- mixer_.reset(new AudioRendererMixer(output_parameters_, sink_));
+ mixer_.reset(
+ new AudioRendererMixer(output_parameters_, sink_, base::Bind(&LogUma)));
mixer_callback_ = sink_->callback();
audio_bus_ = AudioBus::Create(output_parameters_);
@@ -88,17 +93,14 @@ class AudioRendererMixerTest
AudioRendererMixer* GetMixer(int owner_id,
const AudioParameters& params,
+ AudioLatency::LatencyType latency,
const std::string& device_id,
const url::Origin& security_origin,
OutputDeviceStatus* device_status) final {
return mixer_.get();
};
- MOCK_METHOD4(ReturnMixer,
- void(int,
- const AudioParameters&,
- const std::string&,
- const url::Origin&));
+ MOCK_METHOD1(ReturnMixer, void(AudioRendererMixer*));
MOCK_METHOD4(
GetOutputDeviceInfo,
@@ -333,7 +335,7 @@ class AudioRendererMixerTest
return new AudioRendererMixerInput(
this,
// Zero frame id, default device ID and security origin.
- 0, std::string(), url::Origin());
+ 0, std::string(), url::Origin(), AudioLatency::LATENCY_PLAYBACK);
}
protected:
« no previous file with comments | « media/base/audio_renderer_mixer_pool.h ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698