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

Unified Diff: media/renderers/audio_renderer_impl_unittest.cc

Issue 2471733004: Revert of Make more media APIs aware of |delay| and |delay_timestamp| (Closed)
Patch Set: Created 4 years, 1 month 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/renderers/audio_renderer_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/renderers/audio_renderer_impl_unittest.cc
diff --git a/media/renderers/audio_renderer_impl_unittest.cc b/media/renderers/audio_renderer_impl_unittest.cc
index e68422cc362984d2d9a1fae811ed818ea907dab0..c5b51afbc155b9fb95a4a7c029f33d798d1243b5 100644
--- a/media/renderers/audio_renderer_impl_unittest.cc
+++ b/media/renderers/audio_renderer_impl_unittest.cc
@@ -317,16 +317,16 @@
// buffer. Returns true if and only if all of |requested_frames| were able
// to be consumed.
bool ConsumeBufferedData(OutputFrames requested_frames,
- base::TimeDelta delay) {
+ uint32_t frames_delayed) {
std::unique_ptr<AudioBus> bus =
AudioBus::Create(kChannels, requested_frames.value);
int frames_read = 0;
- EXPECT_TRUE(sink_->Render(bus.get(), delay, &frames_read));
+ EXPECT_TRUE(sink_->Render(bus.get(), frames_delayed, &frames_read));
return frames_read == requested_frames.value;
}
bool ConsumeBufferedData(OutputFrames requested_frames) {
- return ConsumeBufferedData(requested_frames, base::TimeDelta());
+ return ConsumeBufferedData(requested_frames, 0);
}
base::TimeTicks ConvertMediaTime(base::TimeDelta timestamp,
@@ -775,7 +775,7 @@
std::unique_ptr<AudioBus> bus = AudioBus::Create(hardware_params_);
int frames_read = 0;
for (int i = 0; i < std::floor(kBuffers); ++i) {
- EXPECT_TRUE(sink_->Render(bus.get(), base::TimeDelta(), &frames_read));
+ EXPECT_TRUE(sink_->Render(bus.get(), 0, &frames_read));
EXPECT_EQ(frames_read, bus->frames());
for (int j = 0; j < bus->frames(); ++j)
ASSERT_FLOAT_EQ(0.0f, bus->channel(0)[j]);
@@ -784,7 +784,7 @@
}
// Verify the last buffer is half silence and half real data.
- EXPECT_TRUE(sink_->Render(bus.get(), base::TimeDelta(), &frames_read));
+ EXPECT_TRUE(sink_->Render(bus.get(), 0, &frames_read));
EXPECT_EQ(frames_read, bus->frames());
const int zero_frames =
bus->frames() * (kBuffers - static_cast<int>(kBuffers));
@@ -803,20 +803,20 @@
// Verify the first buffer is real data.
int frames_read = 0;
std::unique_ptr<AudioBus> bus = AudioBus::Create(hardware_params_);
- EXPECT_TRUE(sink_->Render(bus.get(), base::TimeDelta(), &frames_read));
+ EXPECT_TRUE(sink_->Render(bus.get(), 0, &frames_read));
EXPECT_NE(0, frames_read);
for (int i = 0; i < bus->frames(); ++i)
ASSERT_NE(0.0f, bus->channel(0)[i]);
// Verify after suspend we get silence.
renderer_->OnSuspend();
- EXPECT_TRUE(sink_->Render(bus.get(), base::TimeDelta(), &frames_read));
+ EXPECT_TRUE(sink_->Render(bus.get(), 0, &frames_read));
EXPECT_EQ(0, frames_read);
// Verify after resume we get audio.
bus->Zero();
renderer_->OnResume();
- EXPECT_TRUE(sink_->Render(bus.get(), base::TimeDelta(), &frames_read));
+ EXPECT_TRUE(sink_->Render(bus.get(), 0, &frames_read));
EXPECT_NE(0, frames_read);
for (int i = 0; i < bus->frames(); ++i)
ASSERT_NE(0.0f, bus->channel(0)[i]);
@@ -1005,7 +1005,7 @@
std::round(delay_frames * kOutputMicrosPerFrame));
frames_to_consume.value = frames_buffered().value / 16;
- EXPECT_TRUE(ConsumeBufferedData(frames_to_consume, delay_time));
+ EXPECT_TRUE(ConsumeBufferedData(frames_to_consume, delay_frames));
// Verify time is adjusted for the current delay.
current_time = tick_clock_->NowTicks() + delay_time;
« no previous file with comments | « media/renderers/audio_renderer_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698