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

content/common/media/audio_messages.h

Index: content/common/media/audio_messages.h
diff --git a/content/common/media/audio_messages.h b/content/common/media/audio_messages.h
index d3df85976fe0decab8fd53e43f511891b8e431ed..4112074315e307b1b46b82f1b33ac764929ad98e 100644
--- a/content/common/media/audio_messages.h
+++ b/content/common/media/audio_messages.h
@@ -44,6 +44,14 @@ 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
@@ -84,20 +92,26 @@ IPC_MESSAGE_CONTROL2(AudioInputMsg_NotifyStreamVolume,
 
 // 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.
@@ -141,9 +155,8 @@ IPC_MESSAGE_CONTROL2(AudioInputHostMsg_SetVolume,
                      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.