Do not pass the string device_id via IPC message to create an audio input stream

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"
 #include "content/browser/renderer_host/media/audio_input_device_manager.h"
 #include "content/browser/renderer_host/media/audio_input_sync_writer.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/browser/renderer_host/media/web_contents_audio_input_stream.h"
 #include "content/browser/renderer_host/media/web_contents_capture_util.h"
 #include "content/common/media/audio_messages.h"
+#include "media/audio/audio_manager_base.h"
 
 namespace content {
 
 struct AudioInputRendererHost::AudioEntry {
   AudioEntry();
   ~AudioEntry();
 
   // The AudioInputController that manages the audio input stream.
   scoped_refptr<media::AudioInputController> controller;
 
@@ skipping 149 matching lines @@
   if (!entry)
     return;
 
   DeleteEntryOnError(entry);
 }
 
 bool AudioInputRendererHost::OnMessageReceived(const IPC::Message& message,
                                                bool* message_was_ok) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP_EX(AudioInputRendererHost, message, *message_was_ok)
-    IPC_MESSAGE_HANDLER(AudioInputHostMsg_StartDevice, OnStartDevice)
     IPC_MESSAGE_HANDLER(AudioInputHostMsg_CreateStream, OnCreateStream)
     IPC_MESSAGE_HANDLER(AudioInputHostMsg_AssociateStreamWithConsumer,
                         OnAssociateStreamWithConsumer)
     IPC_MESSAGE_HANDLER(AudioInputHostMsg_RecordStream, OnRecordStream)
     IPC_MESSAGE_HANDLER(AudioInputHostMsg_CloseStream, OnCloseStream)
     IPC_MESSAGE_HANDLER(AudioInputHostMsg_SetVolume, OnSetVolume)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP_EX()
 
   return handled;
 }
 
-void AudioInputRendererHost::OnStartDevice(int stream_id, int session_id) {
-  VLOG(1) << "AudioInputRendererHost::OnStartDevice(stream_id="
-          << 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);
-}
-
 void AudioInputRendererHost::OnCreateStream(
-    int stream_id, const media::AudioParameters& params,
-    const std::string& device_id, bool automatic_gain_control) {
+    int stream_id, int session_id, const media::AudioParameters& params,
+    bool automatic_gain_control) {
   VLOG(1) << "AudioInputRendererHost::OnCreateStream(stream_id="
-          << stream_id << ")";
+          << stream_id << ", session_id=" << session_id << ")";
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   // media::AudioParameters is validated in the deserializer.
   if (LookupById(stream_id) != NULL) {
     SendErrorMessage(stream_id);
     return;
   }
 
+  // Check if we have the permission to open the device and which device to use.
+  std::string device_id = media::AudioManagerBase::kDefaultDeviceId;
+  if (session_id !=  AudioInputDeviceManager::kFakeOpenSessionId) {

[palmer, 2013/03/12 17:45:11]

NIT: Two spaces after the "!=". Just use one.

[no longer working on chromium, 2013/03/14 10:47:32]

Done.

+    const StreamDeviceInfo& info = media_stream_manager_->
+        audio_input_device_manager()->GetOpenedDeviceInfoById(session_id);
+    if (info.device.id.empty()) {
+      // An empty device info indicates that no permission has been granted .

[palmer, 2013/03/12 17:45:11]

NIT: Space before period.

[no longer working on chromium, 2013/03/14 10:47:32]

Done.

+      SendErrorMessage(stream_id);
+      DLOG(WARNING) << "No permission has been granted to input stream with "
+                    << "session_id=" << session_id;
+      return;
+    }
+
+    device_id = info.device.id;
+  }
+
   media::AudioParameters audio_params(params);
-
   if (media_stream_manager_->audio_input_device_manager()->
       ShouldUseFakeDevice()) {
     audio_params.Reset(media::AudioParameters::AUDIO_FAKE,
                        params.channel_layout(), 0, params.sample_rate(),
                        params.bits_per_sample(), params.frames_per_buffer());
   }
 
   uint32 buffer_size = audio_params.GetBytesPerBuffer();
 
   // Create a new AudioEntry structure.
@@ skipping 76 matching lines @@
   entry->controller->Record();
 }
 
 void AudioInputRendererHost::OnCloseStream(int stream_id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   AudioEntry* entry = LookupById(stream_id);
 
   if (entry)
     CloseAndDeleteStream(entry);
-
-  int session_id = LookupSessionById(stream_id);
-
-  if (session_id)
-    StopAndDeleteDevice(session_id);
 }
 
 void AudioInputRendererHost::OnSetVolume(int stream_id, double volume) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   AudioEntry* entry = LookupById(stream_id);
   if (!entry) {
     SendErrorMessage(stream_id);
     return;
   }
 
   entry->controller->SetVolume(volume);
 }
 
 void AudioInputRendererHost::SendErrorMessage(int stream_id) {
   Send(new AudioInputMsg_NotifyStreamStateChanged(
       stream_id, media::AudioInputIPCDelegate::kError));
 }
 
 void AudioInputRendererHost::DeleteEntries() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   for (AudioEntryMap::iterator i = audio_entries_.begin();
        i != audio_entries_.end(); ++i) {
     CloseAndDeleteStream(i->second);
   }
 }
 
-void AudioInputRendererHost::OnDeviceStarted(
-    int session_id, const std::string& device_id) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-  SessionEntryMap::iterator it = session_entries_.find(session_id);
-  if (it == session_entries_.end()) {
-    DLOG(WARNING) << "AudioInputRendererHost::OnDeviceStarted()"
-        " session does not exist.";
-    return;
-  }
-
-  // Notify the renderer with the id of the opened device.
-  Send(new AudioInputMsg_NotifyDeviceStarted(it->second, device_id));
-}
-
-void AudioInputRendererHost::OnDeviceStopped(int session_id) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-
-  SessionEntryMap::iterator it = session_entries_.find(session_id);
-  // Return if the stream has been closed.
-  if (it == session_entries_.end())
-    return;
-
-  int stream_id = it->second;
-  AudioEntry* entry = LookupById(stream_id);
-
-  if (entry) {
-    // Device has been stopped, close the input stream.
-    CloseAndDeleteStream(entry);
-    // Notify the renderer that the state of the input stream has changed.
-    Send(new AudioInputMsg_NotifyStreamStateChanged(
-        stream_id, media::AudioInputIPCDelegate::kStopped));
-  }
-
-  // Delete the session entry.
-  session_entries_.erase(it);
-}
-
-void AudioInputRendererHost::StopAndDeleteDevice(int session_id) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-
-  media_stream_manager_->audio_input_device_manager()->Stop(session_id);
-
-  // Delete the session entry.
-  session_entries_.erase(session_id);
-}
-
 void AudioInputRendererHost::CloseAndDeleteStream(AudioEntry* entry) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   if (!entry->pending_close) {
     entry->controller->Close(base::Bind(&AudioInputRendererHost::DeleteEntry,
                                         this, entry));
     entry->pending_close = true;
   }
 }
 
@@ skipping 33 matching lines @@
   // Iterate the map of entries.
   // TODO(hclam): Implement a faster look up method.
   for (AudioEntryMap::iterator i = audio_entries_.begin();
        i != audio_entries_.end(); ++i) {
     if (controller == i->second->controller.get())
       return i->second;
   }
   return NULL;
 }
 
-int AudioInputRendererHost::LookupSessionById(int stream_id) {
-  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;
-}
-
 }  // namespace content