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

media/mojo/interfaces/media_types.mojom

 // Copyright 2014 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.
 
 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 {
   HAVE_NOTHING,
   HAVE_ENOUGH,
 };
 
 // See media/base/audio_decoder_config.h for descriptions.
 // Kept in sync with media::AudioCodec via static_asserts.
 enum AudioCodec {
@@ skipping 154 matching lines @@
 
 // See media/base/encryption_scheme.h.
 // Kept in sync with media::CipherMode via static_asserts.
 enum CipherMode {
   UNENCRYPTED = 0,
   AES_CTR,
   AES_CBC,
   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 {
   uint32 encrypt_blocks;
   uint32 skip_blocks;
 };
 
 // This defines a mojo transport format for media::EncryptionScheme.
 // See media/base/encryption_scheme.h for description.
 struct EncryptionScheme {
@@ skipping 124 matching lines @@
 
   // Stride and offsets for each plane. Offsets are relative to the start
   // of |frame_data|.
   int32 y_stride;
   int32 u_stride;
   int32 v_stride;
   uint64 y_offset;
   uint64 u_offset;
   uint64 v_offset;
 };