Index: media/audio/win/audio_output_win_unittest.cc |
diff --git a/media/audio/win/audio_output_win_unittest.cc b/media/audio/win/audio_output_win_unittest.cc |
index 68f61eaf4d7395b3b91e4074e7f8fe150778e4be..10284883c5b04d36006b510db43f92b18f2d8482 100644 |
--- a/media/audio/win/audio_output_win_unittest.cc |
+++ b/media/audio/win/audio_output_win_unittest.cc |
@@ -32,7 +32,9 @@ using ::testing::Return; |
namespace media { |
-static int ClearData(AudioBus* audio_bus, uint32 total_bytes_delay) { |
+static int ClearData(AudioBus* audio_bus, |
+ uint32_t total_bytes_delay, |
+ uint32_t frames_skipped) { |
audio_bus->Zero(); |
return audio_bus->frames(); |
} |
@@ -46,7 +48,9 @@ class TestSourceBasic : public AudioOutputStream::AudioSourceCallback { |
had_error_(0) { |
} |
// AudioSourceCallback::OnMoreData implementation: |
- int OnMoreData(AudioBus* audio_bus, uint32 total_bytes_delay) override { |
+ int OnMoreData(AudioBus* audio_bus, |
+ uint32_t total_bytes_delay, |
+ uint32_t frames_skipped) override { |
++callback_count_; |
// Touch the channel memory value to make sure memory is good. |
audio_bus->Zero(); |
@@ -80,9 +84,11 @@ class TestSourceLaggy : public TestSourceBasic { |
explicit TestSourceLaggy(int lag_in_ms) |
: lag_in_ms_(lag_in_ms) { |
} |
- int OnMoreData(AudioBus* audio_bus, uint32 total_bytes_delay) override { |
+ int OnMoreData(AudioBus* audio_bus, |
+ uint32_t total_bytes_delay, |
+ uint32_t frames_skipped) override { |
// Call the base, which increments the callback_count_. |
- TestSourceBasic::OnMoreData(audio_bus, total_bytes_delay); |
+ TestSourceBasic::OnMoreData(audio_bus, total_bytes_delay, frames_skipped); |
if (callback_count() > kMaxNumBuffers) { |
::Sleep(lag_in_ms_); |
} |
@@ -474,22 +480,21 @@ TEST(WinAudioTest, PCMWaveStreamPendingBytes) { |
// pending bytes will go down and eventually read zero. |
InSequence s; |
- EXPECT_CALL(source, OnMoreData(NotNull(), 0)) |
- .WillOnce(Invoke(ClearData)); |
+ EXPECT_CALL(source, OnMoreData(NotNull(), 0, 0)).WillOnce(Invoke(ClearData)); |
// Note: If AudioManagerWin::NumberOfWaveOutBuffers() ever changes, or if this |
// test is run on Vista, these expectations will fail. |
- EXPECT_CALL(source, OnMoreData(NotNull(), bytes_100_ms)) |
+ EXPECT_CALL(source, OnMoreData(NotNull(), bytes_100_ms, 0)) |
.WillOnce(Invoke(ClearData)); |
- EXPECT_CALL(source, OnMoreData(NotNull(), 2 * bytes_100_ms)) |
+ EXPECT_CALL(source, OnMoreData(NotNull(), 2 * bytes_100_ms, 0)) |
.WillOnce(Invoke(ClearData)); |
- EXPECT_CALL(source, OnMoreData(NotNull(), 2 * bytes_100_ms)) |
+ EXPECT_CALL(source, OnMoreData(NotNull(), 2 * bytes_100_ms, 0)) |
.Times(AnyNumber()) |
.WillRepeatedly(Return(0)); |
- EXPECT_CALL(source, OnMoreData(NotNull(), bytes_100_ms)) |
+ EXPECT_CALL(source, OnMoreData(NotNull(), bytes_100_ms, 0)) |
.Times(AnyNumber()) |
.WillRepeatedly(Return(0)); |
- EXPECT_CALL(source, OnMoreData(NotNull(), 0)) |
+ EXPECT_CALL(source, OnMoreData(NotNull(), 0, 0)) |
.Times(AnyNumber()) |
.WillRepeatedly(Return(0)); |
@@ -514,7 +519,9 @@ class SyncSocketSource : public AudioOutputStream::AudioSourceCallback { |
~SyncSocketSource() override {} |
// AudioSourceCallback::OnMoreData implementation: |
- int OnMoreData(AudioBus* audio_bus, uint32 total_bytes_delay) override { |
+ int OnMoreData(AudioBus* audio_bus, |
+ uint32_t total_bytes_delay, |
+ uint32_t frames_skipped) override { |
socket_->Send(&total_bytes_delay, sizeof(total_bytes_delay)); |
uint32 size = socket_->Receive(data_.get(), data_size_); |
DCHECK_EQ(static_cast<size_t>(size) % sizeof(*audio_bus_->channel(0)), 0U); |
@@ -565,7 +572,7 @@ DWORD __stdcall SyncSocketThread(void* context) { |
if (ctx.socket->Receive(&total_bytes_delay, sizeof(total_bytes_delay)) == 0) |
break; |
if ((times > 0) && (total_bytes_delay < 1000)) __debugbreak(); |
- sine.OnMoreData(audio_bus.get(), total_bytes_delay); |
+ sine.OnMoreData(audio_bus.get(), total_bytes_delay, 0); |
ctx.socket->Send(data.get(), ctx.packet_size_bytes); |
++times; |
} |