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

Unified Diff: content/browser/renderer_host/media/audio_renderer_host.cc

Issue 11166002: Plumb render view ID from audio-related code in renderer through IPCs to AudioRendererHost in brows… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased. Created 8 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
Index: content/browser/renderer_host/media/audio_renderer_host.cc
diff --git a/content/browser/renderer_host/media/audio_renderer_host.cc b/content/browser/renderer_host/media/audio_renderer_host.cc
index 478899c343a8dda61c25d2a1cb4593008640a073..390652a47a3f7a7d9cb713ed1825181ab98de668 100644
--- a/content/browser/renderer_host/media/audio_renderer_host.cc
+++ b/content/browser/renderer_host/media/audio_renderer_host.cc
@@ -194,11 +194,25 @@ bool AudioRendererHost::OnMessageReceived(const IPC::Message& message,
return handled;
}
+// TODO(miu): AudioRendererHost will keep track of the render_view_id associated
+// with the created audio stream in a soon-upcoming change.
void AudioRendererHost::OnCreateStream(
- int stream_id, const media::AudioParameters& params, int input_channels) {
+ int render_view_id, int stream_id, const media::AudioParameters& params,
+ int input_channels) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
DCHECK(LookupById(stream_id) == NULL);
+ // TODO(miu): There is still code in the renderer which has been stubbed out,
+ // and should be implemented to plumb in the source render_view_id. See
+ // http://crbug.com/156535. In debug mode, log a WARNING until this is
+ // resolved.
+ DLOG_IF(WARNING, render_view_id <= 0)
+ << ("FIXME: AudioRendererHost::OnCreateStream called with unknown render "
+ "view.");
+ DVLOG(1) << "AudioRendererHost@" << this
+ << "::OnCreateStream(render_view_id=" << render_view_id
+ << ", stream_id=" << stream_id << ")";
+
media::AudioParameters audio_params(params);
uint32 buffer_size = media::AudioBus::CalculateMemorySize(audio_params);
DCHECK_GT(buffer_size, 0U);

Powered by Google App Engine
This is Rietveld 408576698