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

Unified Diff: content/renderer/media/renderer_webaudiodevice_impl.cc

Issue 2268253002: UMA stats for browser/renderer audio rendering buffer size mismatch. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: histogram rename Created 4 years, 4 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: content/renderer/media/renderer_webaudiodevice_impl.cc
diff --git a/content/renderer/media/renderer_webaudiodevice_impl.cc b/content/renderer/media/renderer_webaudiodevice_impl.cc
index b1f80371acdd048abb8336536090bc2688ea049c..562f2e1c633ad98ca0ffc6b42a4d1151b9411a53 100644
--- a/content/renderer/media/renderer_webaudiodevice_impl.cc
+++ b/content/renderer/media/renderer_webaudiodevice_impl.cc
@@ -76,7 +76,13 @@ void RendererWebAudioDeviceImpl::start() {
AudioDeviceFactory::kSourceWebAudioInteractive,
render_frame ? render_frame->GetRoutingID() : MSG_ROUTING_NONE,
session_id_, std::string(), security_origin_);
- sink_->Initialize(params_, this);
+
+ // Specify the latency info to be passed to the browser side.
+ media::AudioParameters sink_params(params_);
+ sink_params.set_latency_tag(AudioDeviceFactory::GetSourceLatencyType(
+ AudioDeviceFactory::kSourceWebAudioInteractive));
+
+ sink_->Initialize(sink_params, this);
// TODO(miu): Remove this temporary instrumentation to root-cause a memory
// use-after-free issue. http://crbug.com/619463
{
« no previous file with comments | « content/renderer/media/audio_renderer_mixer_manager.cc ('k') | content/renderer/media/track_audio_renderer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698