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

Unified Diff: media/audio/android/audio_record_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/audio_record_input.cc
diff --git a/media/audio/android/audio_record_input.cc b/media/audio/android/audio_record_input.cc
index 8e436ff328b00606437f92b32cdaa2b5c5d800e7..2742a9d7d4819845017292d08402f17de31b3173 100644
--- a/media/audio/android/audio_record_input.cc
+++ b/media/audio/android/audio_record_input.cc
@@ -56,15 +56,17 @@ bool AudioRecordInputStream::RegisterAudioRecordInput(JNIEnv* env) {
void AudioRecordInputStream::OnData(JNIEnv* env,
const JavaParamRef<jobject>& obj,
jint size,
- jint hardware_delay_bytes) {
+ jint hardware_delay_ms) {
DCHECK(direct_buffer_address_);
DCHECK_EQ(size,
audio_bus_->frames() * audio_bus_->channels() * bytes_per_sample_);
// Passing zero as the volume parameter indicates there is no access to a
// hardware volume slider.
- audio_bus_->FromInterleaved(
- direct_buffer_address_, audio_bus_->frames(), bytes_per_sample_);
- callback_->OnData(this, audio_bus_.get(), hardware_delay_bytes, 0.0);
+ audio_bus_->FromInterleaved(direct_buffer_address_, audio_bus_->frames(),
+ bytes_per_sample_);
+ callback_->OnData(this, audio_bus_.get(),
+ base::TimeDelta::FromMilliseconds(hardware_delay_ms),
+ base::TimeTicks::Now(), 0.0);
}
bool AudioRecordInputStream::Open() {

Powered by Google App Engine
This is Rietveld 408576698