Audio codec usage statistics

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

Index: webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.cc
diff --git a/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.cc b/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.cc
index e4da8d7440bc5520342474ca6e9a9360f885c9bc..e5c513dc7dfe21180a41db67c3234d5267640721 100644
--- a/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.cc
+++ b/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.cc
@@ -116,6 +116,10 @@ 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)) {}
 
@@ -129,4 +133,8 @@ 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