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

Unified Diff: media/audio/android/opensles_input.cc

Issue 2689483006: Switch browser side audio capture path to use base time primitives. (Closed)
Patch Set: Bloop Created 3 years, 10 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
Index: media/audio/android/opensles_input.cc
diff --git a/media/audio/android/opensles_input.cc b/media/audio/android/opensles_input.cc
index 748a9cca80285991e2472c92d5e450d110295cfd..6a943ce88767dd902b988d618fef157409c30025 100644
--- a/media/audio/android/opensles_input.cc
+++ b/media/audio/android/opensles_input.cc
@@ -42,6 +42,8 @@ OpenSLESInputStream::OpenSLESInputStream(AudioManagerAndroid* audio_manager,
format_.channelMask = ChannelCountToSLESChannelMask(params.channels());
buffer_size_bytes_ = params.GetBytesPerBuffer();
+ hardware_delay_ = base::TimeDelta::FromSecondsD(
+ params.frames_per_buffer() / static_cast<double>(params.sample_rate()));
memset(&audio_data_, 0, sizeof(audio_data_));
}
@@ -306,7 +308,8 @@ void OpenSLESInputStream::ReadBufferQueue() {
// TODO(henrika): Investigate if it is possible to get an accurate
// delay estimation.
- callback_->OnData(this, audio_bus_.get(), buffer_size_bytes_, 0.0);
+ callback_->OnData(this, audio_bus_.get(), hardware_delay_,
+ base::TimeTicks::Now(), 0.0);
// Done with this buffer. Send it to device for recording.
SLresult err =

Powered by Google App Engine
This is Rietveld 408576698