Allow AudioOutputDevice objects to be initialized with a specific hardware output device and store …

content/common/media/audio_messages.h

 // 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.
 
 // IPC messages for the audio.
 // Multiply-included message file, hence no include guard.
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 26 matching lines @@
 IPC_ENUM_TRAITS_MAX_VALUE(media::ChannelLayout, media::CHANNEL_LAYOUT_MAX)
 
 IPC_STRUCT_BEGIN(AudioInputHostMsg_CreateStream_Config)
   IPC_STRUCT_MEMBER(media::AudioParameters, params)
   IPC_STRUCT_MEMBER(bool, automatic_gain_control)
   IPC_STRUCT_MEMBER(uint32, shared_memory_count)
 IPC_STRUCT_END()
 
 // Messages sent from the browser to the renderer.
 
+// Tell the renderer process that an audio output device has been authorized
+// for a given stream. The renderer is given the output parameters for the
+// authorized device.
+IPC_MESSAGE_CONTROL3(AudioMsg_NotifyDeviceAuthorized,
+                     int /* stream id */,
+                     bool /* success */,
+                     media::AudioParameters /* output parameters */)
+
 // Tell the renderer process that an audio stream has been created.
 // The renderer process is given a shared memory handle for the audio data
 // buffer it shares with the browser process. It is also given a SyncSocket that
 // it uses to communicate with the browser process about the state of the
 // buffered audio data.
 IPC_MESSAGE_CONTROL4(
    AudioMsg_NotifyStreamCreated,
    int /* stream id */,
    base::SharedMemoryHandle /* handle */,
    base::SyncSocket::TransitDescriptor /* socket descriptor */,
@@ skipping 20 matching lines @@
 IPC_MESSAGE_CONTROL2(AudioInputMsg_NotifyStreamStateChanged,
                      int /* stream id */,
                      media::AudioInputIPCDelegateState /* new state */)
 
 IPC_MESSAGE_CONTROL2(AudioInputMsg_NotifyStreamVolume,
                      int /* stream id */,
                      double /* volume */)
 
 // Notification message sent from AudioRendererHost to renderer for state
 // update after the renderer has requested a SwitchOutputDevice.
-IPC_MESSAGE_CONTROL3(AudioMsg_NotifyOutputDeviceSwitched,
+IPC_MESSAGE_CONTROL2(AudioMsg_NotifyOutputDeviceSwitched,
                      int /* stream id */,
-                     int /* request id */,
                      media::SwitchOutputDeviceResult /* result */)
 
 // Messages sent from the renderer to the browser.
 
-// Request that is sent to the browser for creating an audio output stream.
-// |render_frame_id| is the routing ID for the RenderFrame producing the audio
-// data.
-IPC_MESSAGE_CONTROL4(AudioHostMsg_CreateStream,
+// Request that is sent to the browser to authorize use of an audio output

[palmer, 2015/09/15 20:29:24]

To request the use of?

[Guido Urdaneta, 2015/09/16 11:34:49]

Done.

+// device. |render_frame_id| is the routing ID for the RenderFrame producing
+// the audio data.
+IPC_MESSAGE_CONTROL5(AudioHostMsg_RequestDeviceAuthorization,
                      int /* stream_id */,
                      int /* render_frame_id */,
                      int /* session_id */,
+                     std::string /* device_id */,

[palmer, 2015/09/15 20:29:24]

What lexical/syntactic form do these device IDs have?

[Guido Urdaneta, 2015/09/16 11:34:49]

It's an opaque ID consisting of a hash of the actual device ID (known only by
the browser) and the security origin. Example value:
"6e18f44f010fdf64599acfe8dba0cc99639a8591c8b8db395360acc0533f3bec"

+                     GURL /* security_origin */)

[palmer, 2015/09/15 20:29:24]

url::Origin is the canonical class for this purpose, now.

[Guido Urdaneta, 2015/09/16 11:34:49]

Done.

+
+// Request that is sent to the browser for creating an audio output stream.
+IPC_MESSAGE_CONTROL3(AudioHostMsg_CreateStream,
+                     int /* stream_id */,
+                     int /* render_frame_id */,
                      media::AudioParameters /* params */)
 
 // Request that is sent to the browser for creating an audio input stream.
 // |render_frame_id| is the routing ID for the RenderFrame consuming the audio
 // data.
 IPC_MESSAGE_CONTROL4(AudioInputHostMsg_CreateStream,
                      int /* stream_id */,
                      int /* render_frame_id */,
                      int /* session_id */,
                      AudioInputHostMsg_CreateStream_Config)
@@ skipping 23 matching lines @@
 IPC_MESSAGE_CONTROL2(AudioHostMsg_SetVolume,
                      int /* stream_id */,
                      double /* volume */)
 
 // Set audio volume of the input stream specified by stream_id.
 IPC_MESSAGE_CONTROL2(AudioInputHostMsg_SetVolume,
                      int /* stream_id */,
                      double /* volume */)
 
 // Switch the output device of the stream specified by stream_id.
-IPC_MESSAGE_CONTROL5(AudioHostMsg_SwitchOutputDevice,
+IPC_MESSAGE_CONTROL4(AudioHostMsg_SwitchOutputDevice,
                      int /* stream_id */,
                      int /* render_frame_id */,
                      std::string /* device_id */,
-                     GURL /* security_origin */,
-                     int /* request_id */)
+                     GURL /* security_origin */)

[palmer, 2015/09/15 20:29:24]

url::Origin here too.

[Guido Urdaneta, 2015/09/16 11:34:49]

Done.