UMA stats for browser/renderer audio rendering buffer size mismatch.

media/base/audio_parameters.cc

 // 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.
 
 #include "media/base/audio_parameters.h"
 
 #include "base/logging.h"
 #include "media/base/limits.h"
 
 namespace media {
 
 AudioParameters::AudioParameters()
     : AudioParameters(AUDIO_PCM_LINEAR, CHANNEL_LAYOUT_NONE, 0, 0, 0) {}
 
 AudioParameters::AudioParameters(Format format,
                                  ChannelLayout channel_layout,
                                  int sample_rate,
                                  int bits_per_sample,
-                                 int frames_per_buffer) {
+                                 int frames_per_buffer)
+    : latency_tag_(AudioLatency::LATENCY_COUNT) {

[Henrik Grunell, 2016/08/23 15:53:19]

I don't think an invalid value should be used to say it's unspecified. What
about making it a base::Optional?

[o1ka, 2016/08/23 16:20:56]

It would be a pain for IPC

[Henrik Grunell, 2016/08/24 12:47:15]

OK. It's still hacky I think to use the count to indicate unspecified. I'd
prefer an explicit "UNSPECIFIED" value. Let's see what the other reviewers say
about the general idea before digging in more.

   Reset(format, channel_layout, sample_rate, bits_per_sample,
         frames_per_buffer);
 }
 
 AudioParameters::~AudioParameters() {}
 
 AudioParameters::AudioParameters(const AudioParameters&) = default;
 AudioParameters& AudioParameters::operator=(const AudioParameters&) = default;
 
 void AudioParameters::Reset(Format format,
@@ skipping 68 matching lines @@
 
 // static
 AudioParameters AudioParameters::UnavailableDeviceParams() {
   return media::AudioParameters(
       media::AudioParameters::AUDIO_FAKE, media::CHANNEL_LAYOUT_STEREO,
       media::AudioParameters::kAudioCDSampleRate, 16,
       media::AudioParameters::kAudioCDSampleRate / 10);
 }
 
 }  // namespace media