Emit Media Source codec names as media_log events.

media/base/audio_decoder_config.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_AUDIO_DECODER_CONFIG_H_
 #define MEDIA_BASE_AUDIO_DECODER_CONFIG_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 27 matching lines @@
   kCodecPCM_ALAW = 14,
   // DO NOT ADD RANDOM AUDIO CODECS!
   //
   // The only acceptable time to add a new codec is if there is production code
   // that uses said codec in the same CL.
 
   // Must always be equal to the largest entry ever logged.
   kAudioCodecMax = kCodecPCM_ALAW,
 };
 
+const char* AudioCodecName(AudioCodec audio_codec, SampleFormat sample_format);

[DaleCurtis, 2014/11/19 00:01:56]

Instead of adding it like this, how about having a
AudioDecoderConfig::GetHumanReadableCodecName() function? The
AsHumanReadableString() method can then use this method too for better logs.

I'd also return std::string like that method too.

[watk, 2014/11/19 19:18:09]

Done.

+
 // TODO(dalecurtis): FFmpeg API uses |bytes_per_channel| instead of
 // |bits_per_channel|, we should switch over since bits are generally confusing
 // to work with.
 class MEDIA_EXPORT AudioDecoderConfig {
  public:
   // Constructs an uninitialized object. Clients should call Initialize() with
   // appropriate values before using.
   AudioDecoderConfig();
 
   // Constructs an initialized object. It is acceptable to pass in NULL for
@@ skipping 67 matching lines @@
   int codec_delay_;
 
   // Not using DISALLOW_COPY_AND_ASSIGN here intentionally to allow the compiler
   // generated copy constructor and assignment operator. Since the extra data is
   // typically small, the performance impact is minimal.
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_AUDIO_DECODER_CONFIG_H_