Plumb render view ID from audio-related code in renderer through IPCs to AudioRendererHost in brows…

content/browser/renderer_host/media/audio_input_renderer_host.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/media/audio_input_renderer_host.h"
 
 #include "base/bind.h"
 #include "base/metrics/histogram.h"
 #include "base/process.h"
 #include "base/shared_memory.h"
@@ skipping 165 matching lines @@
           << stream_id << ", session_id = " << session_id << ")";
 
   // Add the session entry to the map.
   session_entries_[session_id] = stream_id;
 
   // Start the device with the session_id. If the device is started
   // successfully, OnDeviceStarted() callback will be triggered.
   media_stream_manager_->audio_input_device_manager()->Start(session_id, this);
 }
 
+// TODO(miu): AudioInputRendererHost will keep track of the render_view_id
+// associated with the created audio stream in a soon-upcoming change.
 void AudioInputRendererHost::OnCreateStream(
-    int stream_id, const media::AudioParameters& params,
+    int render_view_id, int stream_id, const media::AudioParameters& params,
     const std::string& device_id, bool automatic_gain_control) {
-  VLOG(1) << "AudioInputRendererHost::OnCreateStream(stream_id="
-          << stream_id << ")";
   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.  In debug
+  // mode, log a WARNING until this is resolved.
+  DLOG_IF(WARNING, render_view_id <= 0)
+      << ("FIXME: AudioInputRendererHost::OnCreateStream called with unknown "
+          "render view.");
+  VLOG(1) << "AudioInputRendererHost@" << this

[Chris Rogers, 2012/10/17 20:43:11]

shouldn't this be a DVLOG?

[miu, 2012/10/17 22:06:30]

Done.

+          << "::OnCreateStream(render_view_id=" << render_view_id
+          << ", stream_id=" << stream_id << ")";
+
   media::AudioParameters audio_params(params);
 
   DCHECK_GT(audio_params.frames_per_buffer(), 0);
   uint32 buffer_size = audio_params.GetBytesPerBuffer();
 
   // Create a new AudioEntry structure.
   scoped_ptr<AudioEntry> entry(new AudioEntry());
 
   uint32 mem_size = sizeof(media::AudioInputBufferParameters) + buffer_size;
 
@@ skipping 196 matching lines @@
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   for (SessionEntryMap::iterator it = session_entries_.begin();
        it != session_entries_.end(); ++it) {
     if (stream_id == it->second) {
       return it->first;
     }
   }
   return 0;
 }