Mojo interfaces needed for switching audio rendering stream creation and closing from IPC to Mojo

media/mojo/interfaces/media_types.mojom

Index: media/mojo/interfaces/media_types.mojom
diff --git a/media/mojo/interfaces/media_types.mojom b/media/mojo/interfaces/media_types.mojom
index 956bf07e53060b9dc1e61ed11d0f6e38e57e8437..753c7fe624c093421234d39e0032895faad4e2e9 100644
--- a/media/mojo/interfaces/media_types.mojom
+++ b/media/mojo/interfaces/media_types.mojom
@@ -6,6 +6,15 @@ module media.interfaces;
 
 import "ui/mojo/geometry/geometry.mojom";
 
+// See media/audio/audio_parameters.h for descriptions.
+// Kept in sync with media::AudioParameters::Format via static_asserts.
+enum AudioFormat {
+  AUDIO_PCM_LINEAR = 0,
+  AUDIO_PCM_LOW_LATENCY,
+  AUDIO_FAKE,
+  AUDIO_FORMAT_LAST = AUDIO_FAKE,
+};
+
 // See media/base/buffering_state.h for descriptions.
 // Kept in sync with media::BufferingState via static_asserts.
 enum BufferingState {
@@ -180,6 +189,18 @@ enum CipherMode {
   MAX = AES_CBC
 };
 
+struct AudioOutputStreamParameters {

[xhwang, 2016/04/26 22:54:42]

Is this the equivalence of media::AudioParameters? If so, we typically keep
using the same name for native type and mojo type for better readability. So
this should also named AudioParameters. Also, please add a comment about the
corresponding native type.

[rchtara, 2016/04/29 12:54:47]

This defines a mojo transport format for media::AudioParameters without the
|mic_positions_| vector. This will be used by media.interfaces.AudioOutput.
CreateStream which doesn't need the vector.
A different name was used to avoid confusion. 

+  AudioFormat format_;                   // Format of the stream.

[xhwang, 2016/04/26 22:54:42]

We don't use trailing "_" for mojo struct members.

[rchtara, 2016/04/29 12:54:47]

Done.

+  ChannelLayout channel_layout_;    // Order of surround sound channels.
+  int32 channels_;                  // Number of channels. Value set based on
+                                    // |channel_layout|.
+  int32 sample_rate_;               // Sampling frequency/rate.
+  int32 bits_per_sample_;           // Number of bits per sample.
+  int32 frames_per_buffer_;         // Number of frames in a buffer.
+  int32 effects_;                   // Bitmask using PlatformEffectsMask.
+

[xhwang, 2016/04/26 22:54:42]

remote empty line.

[rchtara, 2016/04/29 12:54:47]

Done.

+};
+
 // This defines a mojo transport format for media::EncryptionScheme::Pattern
 // See media/base/encryption_scheme.h for description.
 struct Pattern {