Add UMA reporting for audio hardware parameters.

media/base/channel_layout.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.
 
 #ifndef MEDIA_BASE_CHANNEL_LAYOUT_H_
 #define MEDIA_BASE_CHANNEL_LAYOUT_H_
 
 #include "media/base/media_export.h"
 
 enum ChannelLayout {
+  // Do not change the order of these values.

[scherkus (not reviewing), 2012/09/12 13:34:50]

can you elaborate / mention UMA?

Instead of copying the comment from sample_rates.h I'd use the one from
pipeline_status.h as it's much more informative (it's not so much the order but
rather reuse / always adding new values)

Bonus points for updating the one in sample_rates.h at the same time :)

[DaleCurtis, 2012/09/12 14:03:52]

Done.

   CHANNEL_LAYOUT_NONE = 0,
-  CHANNEL_LAYOUT_UNSUPPORTED,
+  CHANNEL_LAYOUT_UNSUPPORTED = 1,
 
   // Front C
-  CHANNEL_LAYOUT_MONO,
+  CHANNEL_LAYOUT_MONO = 2,
 
   // Front L, Front R
-  CHANNEL_LAYOUT_STEREO,
+  CHANNEL_LAYOUT_STEREO = 3,
 
   // Front L, Front R, Back C
-  CHANNEL_LAYOUT_2_1,
+  CHANNEL_LAYOUT_2_1 = 4,
 
   // Front L, Front R, Front C
-  CHANNEL_LAYOUT_SURROUND,
+  CHANNEL_LAYOUT_SURROUND = 5,
 
   // Front L, Front R, Front C, Back C
-  CHANNEL_LAYOUT_4_0,
+  CHANNEL_LAYOUT_4_0 = 6,
 
   // Front L, Front R, Side L, Side R
-  CHANNEL_LAYOUT_2_2,
+  CHANNEL_LAYOUT_2_2 = 7,
 
   // Front L, Front R, Back L, Back R
-  CHANNEL_LAYOUT_QUAD,
+  CHANNEL_LAYOUT_QUAD = 8,
 
   // Front L, Front R, Front C, Side L, Side R
-  CHANNEL_LAYOUT_5_0,
+  CHANNEL_LAYOUT_5_0 = 9,
 
   // Front L, Front R, Front C, Side L, Side R, LFE
-  CHANNEL_LAYOUT_5_1,
+  CHANNEL_LAYOUT_5_1 = 10,
 
   // Front L, Front R, Front C, Back L, Back R
-  CHANNEL_LAYOUT_5_0_BACK,
+  CHANNEL_LAYOUT_5_0_BACK = 11,
 
   // Front L, Front R, Front C, Back L, Back R, LFE
-  CHANNEL_LAYOUT_5_1_BACK,
+  CHANNEL_LAYOUT_5_1_BACK = 12,
 
   // Front L, Front R, Front C, Side L, Side R, Back L, Back R
-  CHANNEL_LAYOUT_7_0,
+  CHANNEL_LAYOUT_7_0 = 13,
 
   // Front L, Front R, Front C, Side L, Side R, LFE, Back L, Back R
-  CHANNEL_LAYOUT_7_1,
+  CHANNEL_LAYOUT_7_1 = 14,
 
   // Front L, Front R, Front C, Back L, Back R, LFE, Front LofC, Front RofC
-  CHANNEL_LAYOUT_7_1_WIDE,
+  CHANNEL_LAYOUT_7_1_WIDE = 15,
 
   // Stereo L, Stereo R
-  CHANNEL_LAYOUT_STEREO_DOWNMIX,
+  CHANNEL_LAYOUT_STEREO_DOWNMIX = 16,
 
   // Total number of layouts.
-  CHANNEL_LAYOUT_MAX
+  CHANNEL_LAYOUT_MAX  // Must always be last!
 };
 
 enum Channels {
   LEFT = 0,
   RIGHT,
   CENTER,
   LFE,
   BACK_LEFT,
   BACK_RIGHT,
   LEFT_OF_CENTER,
@@ skipping 12 matching lines @@
 // position (because the order is L, R, C, LFE, LS, RS), so
 // kChannelOrderings[CHANNEL_LAYOUT_5POINT1][SIDE_LEFT] = 4;
 // Values of -1 mean the channel at that index is not used for that layout.
 MEDIA_EXPORT extern const int
 kChannelOrderings[CHANNEL_LAYOUT_MAX][CHANNELS_MAX];
 
 // Returns the number of channels in a given ChannelLayout.
 MEDIA_EXPORT int ChannelLayoutToChannelCount(ChannelLayout layout);
 
 #endif  // MEDIA_BASE_CHANNEL_LAYOUT_H_