Audio codec usage statistics

webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.cc

 /*
  *  Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 98 matching lines @@
 size_t AudioEncoderPcmA::EncodeCall(const int16_t* audio,
                                     size_t input_len,
                                     uint8_t* encoded) {
   return WebRtcG711_EncodeA(audio, input_len, encoded);
 }
 
 size_t AudioEncoderPcmA::BytesPerSample() const {
   return 1;
 }
 
+const char* AudioEncoderPcmA::GetCodecName() const {
+  return "g711";

[hlundin-webrtc, 2016/05/11 13:03:55]

g711A, or rather PCMA, per my earlier comments.

[aleloi, 2016/05/11 14:10:49]

Is this one of the codecs we are interested in having statistics on? Meaning,
should I put "g711A" into the histogram_id table and update histograms.xml in
chromium?

[hlundin-webrtc, 2016/05/12 12:23:39]

Yes.

+}
+
 AudioEncoderPcmU::AudioEncoderPcmU(const CodecInst& codec_inst)
     : AudioEncoderPcmU(CreateConfig<AudioEncoderPcmU>(codec_inst)) {}
 
 size_t AudioEncoderPcmU::EncodeCall(const int16_t* audio,
                                     size_t input_len,
                                     uint8_t* encoded) {
   return WebRtcG711_EncodeU(audio, input_len, encoded);
 }
 
 size_t AudioEncoderPcmU::BytesPerSample() const {
   return 1;
 }
 
+const char* AudioEncoderPcmU::GetCodecName() const {
+  return "g711";

[hlundin-webrtc, 2016/05/11 13:03:55]

g711u, or rather PCMU.

[aleloi, 2016/05/11 14:10:49]

Same question as for g711A

[hlundin-webrtc, 2016/05/12 12:23:39]

Yes.

+}
+
 }  // namespace webrtc