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

Unified Diff: media/audio/win/audio_unified_win_unittest.cc

Issue 12611030: Remove unused parameter to OnError() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix nits + rebase Created 7 years, 9 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/audio/win/audio_unified_win.cc ('k') | media/audio/win/wavein_input_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/win/audio_unified_win_unittest.cc
diff --git a/media/audio/win/audio_unified_win_unittest.cc b/media/audio/win/audio_unified_win_unittest.cc
index 9a8e86603735940c1dc9da6598ec9c8c1baaa883..8907612fe409f0e9075da9d7631a9d07902327f0 100644
--- a/media/audio/win/audio_unified_win_unittest.cc
+++ b/media/audio/win/audio_unified_win_unittest.cc
@@ -52,7 +52,7 @@ class MockUnifiedSourceCallback
MOCK_METHOD3(OnMoreIOData, int(AudioBus* source,
AudioBus* dest,
AudioBuffersState buffers_state));
- MOCK_METHOD2(OnError, void(AudioOutputStream* stream, int code));
+ MOCK_METHOD1(OnError, void(AudioOutputStream* stream));
};
// AudioOutputStream::AudioSourceCallback implementation which enables audio
@@ -109,7 +109,7 @@ class UnifiedSourceCallback : public AudioOutputStream::AudioSourceCallback {
return source->frames();
};
- virtual void OnError(AudioOutputStream* stream, int code) {
+ virtual void OnError(AudioOutputStream* stream) {
NOTREACHED();
}
@@ -198,7 +198,7 @@ TEST(WASAPIUnifiedStreamTest, OpenStartAndClose) {
WASAPIUnifiedStream* wus = ausw.Create();
EXPECT_TRUE(wus->Open());
- EXPECT_CALL(source, OnError(wus, _))
+ EXPECT_CALL(source, OnError(wus))
.Times(0);
EXPECT_CALL(source, OnMoreIOData(NotNull(), NotNull(), _))
.Times(Between(0, 1))
@@ -224,7 +224,7 @@ TEST(WASAPIUnifiedStreamTest, StartLoopbackAudio) {
AudioBuffersState min_total_audio_delay(0, 2 * ausw.bytes_per_buffer());
EXPECT_TRUE(wus->Open());
- EXPECT_CALL(source, OnError(wus, _))
+ EXPECT_CALL(source, OnError(wus))
.Times(0);
EXPECT_CALL(source, OnMoreIOData(
NotNull(), NotNull(), DelayGreaterThan(min_total_audio_delay)))
« no previous file with comments | « media/audio/win/audio_unified_win.cc ('k') | media/audio/win/wavein_input_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698