Add a histogram: Media.AVDA.NumAVDAInstances

media/gpu/android_video_decode_accelerator.cc

 // Copyright (c) 2013 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/gpu/android_video_decode_accelerator.h"
 
 #include <stddef.h>
 
 #include <memory>
 
@@ skipping 176 matching lines @@
     // the thread might already be started even if there are no avda instances.
     // Plus, sometimes we just fail to start the thread.
     if (!construction_thread_.IsRunning()) {
       if (!construction_thread_.Start()) {
         LOG(ERROR) << "Failed to start construction thread.";
         return false;
       }
     }
 
     thread_avda_instances_.insert(avda_instance);
+    UMA_HISTOGRAM_CUSTOM_COUNTS("Media.AVDA.NumAVDAInstances",

[rkaplow, 2016/07/29 20:31:58]

This would be better by using the ENUM macro, like 
 UMA_HISTOGRAM_ENUMERATION(name, sample, 21)

since the default numeric macro does exponential sized buckets.

[watk, 2016/08/01 18:09:52]

Good to know. Done.

+                                thread_avda_instances_.size(), 1, 20, 20);
     return true;
   }
 
   // |avda_instance| will no longer need the construction thread.  Stop the
   // thread if this is the last instance.
   void StopThread(AndroidVideoDecodeAccelerator* avda_instance) {
     DCHECK(thread_checker_.CalledOnValidThread());
 
     thread_avda_instances_.erase(avda_instance);
     if (!thread_avda_instances_.empty())
@@ skipping 1565 matching lines @@
 
 bool AndroidVideoDecodeAccelerator::IsMediaCodecSoftwareDecodingForbidden()
     const {
   // Prevent MediaCodec from using its internal software decoders when we have
   // more secure and up to date versions in the renderer process.
   return !config_.is_encrypted && (codec_config_->codec_ == media::kCodecVP8 ||
                                    codec_config_->codec_ == media::kCodecVP9);
 }
 
 }  // namespace media