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

Unified Diff: media/audio/cras/cras_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/cras/cras_input.cc
diff --git a/media/audio/cras/cras_input.cc b/media/audio/cras/cras_input.cc
index b5ac5012ff3ecb773ad3de60d24f2dd137c44466..8add8cbd226768b420aab581f95ba1ea0a0109eb 100644
--- a/media/audio/cras/cras_input.cc
+++ b/media/audio/cras/cras_input.cc
@@ -297,10 +297,6 @@ void CrasInputStream::ReadAudio(size_t frames,
double latency_usec =
latency_ts.tv_sec * base::Time::kMicrosecondsPerSecond +
latency_ts.tv_nsec / base::Time::kNanosecondsPerMicrosecond;
- double frames_latency =
- latency_usec * params_.sample_rate() / base::Time::kMicrosecondsPerSecond;
- unsigned int bytes_latency =
- static_cast<unsigned int>(frames_latency * bytes_per_frame_);
// Update the AGC volume level once every second. Note that, |volume| is
// also updated each time SetVolume() is called through IPC by the
@@ -310,7 +306,9 @@ void CrasInputStream::ReadAudio(size_t frames,
audio_bus_->FromInterleaved(
buffer, audio_bus_->frames(), params_.bits_per_sample() / 8);
- callback_->OnData(this, audio_bus_.get(), bytes_latency, normalized_volume);
+ callback_->OnData(this, audio_bus_.get(),
+ base::TimeDelta::FromMicroseconds(latency_usec),
+ base::TimeTicks::Now(), normalized_volume);
}
void CrasInputStream::NotifyStreamError(int err) {

Powered by Google App Engine
This is Rietveld 408576698