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

Unified Diff: media/blink/webaudiosourceprovider_impl.cc

Issue 2452183003: Revert of Make more media APIs aware of |delay| and |delay_timestamp| (Closed)
Patch Set: Created 4 years, 2 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/silent_sink_suspender_unittest.cc ('k') | media/blink/webaudiosourceprovider_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webaudiosourceprovider_impl.cc
diff --git a/media/blink/webaudiosourceprovider_impl.cc b/media/blink/webaudiosourceprovider_impl.cc
index f892ba63d5fa3ae7cb319635c58bc5222f8695c4..e54dbc81aa1f4e9d37989d16385b47f8707dae10 100644
--- a/media/blink/webaudiosourceprovider_impl.cc
+++ b/media/blink/webaudiosourceprovider_impl.cc
@@ -11,7 +11,6 @@
#include "base/logging.h"
#include "base/macros.h"
#include "base/memory/ptr_util.h"
-#include "media/base/audio_timestamp_helper.h"
#include "media/base/bind_to_current_loop.h"
#include "third_party/WebKit/public/platform/WebAudioSourceProviderClient.h"
@@ -70,10 +69,9 @@
// AudioRendererSink::RenderCallback implementation.
// These are forwarders to |renderer_| and are here to allow for a client to
// get a copy of the rendered audio by SetCopyAudioCallback().
- int Render(base::TimeDelta delay,
- base::TimeTicks delay_timestamp,
- int prior_frames_skipped,
- AudioBus* dest) override;
+ int Render(AudioBus* audio_bus,
+ uint32_t frames_delayed,
+ uint32_t frames_skipped) override;
void OnRenderError() override;
bool IsInitialized() const { return !!renderer_; }
@@ -158,8 +156,7 @@
DCHECK(client_);
DCHECK_EQ(tee_filter_->channels(), bus_wrapper_->channels());
- const int frames = tee_filter_->Render(
- base::TimeDelta(), base::TimeTicks::Now(), 0, bus_wrapper_.get());
+ const int frames = tee_filter_->Render(bus_wrapper_.get(), 0, 0);
if (frames < incoming_number_of_frames)
bus_wrapper_->ZeroFramesPartial(frames, incoming_number_of_frames - frames);
@@ -257,8 +254,7 @@
}
int WebAudioSourceProviderImpl::RenderForTesting(AudioBus* audio_bus) {
- return tee_filter_->Render(base::TimeDelta(), base::TimeTicks::Now(), 0,
- audio_bus);
+ return tee_filter_->Render(audio_bus, 0, 0);
}
void WebAudioSourceProviderImpl::OnSetFormat() {
@@ -270,19 +266,15 @@
client_->setFormat(tee_filter_->channels(), tee_filter_->sample_rate());
}
-int WebAudioSourceProviderImpl::TeeFilter::Render(
- base::TimeDelta delay,
- base::TimeTicks delay_timestamp,
- int prior_frames_skipped,
- AudioBus* audio_bus) {
+int WebAudioSourceProviderImpl::TeeFilter::Render(AudioBus* audio_bus,
+ uint32_t frames_delayed,
+ uint32_t frames_skipped) {
DCHECK(IsInitialized());
- const int num_rendered_frames = renderer_->Render(
- delay, delay_timestamp, prior_frames_skipped, audio_bus);
+ const int num_rendered_frames =
+ renderer_->Render(audio_bus, frames_delayed, frames_skipped);
if (!copy_audio_bus_callback_.is_null()) {
- const int64_t frames_delayed =
- AudioTimestampHelper::TimeToFrames(delay, sample_rate_);
std::unique_ptr<AudioBus> bus_copy =
AudioBus::Create(audio_bus->channels(), audio_bus->frames());
audio_bus->CopyTo(bus_copy.get());
« no previous file with comments | « media/base/silent_sink_suspender_unittest.cc ('k') | media/blink/webaudiosourceprovider_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698