PerformanceMonitor: Add a new UMA histograms to track average CPU utilization

chrome/browser/performance_monitor/process_metrics_history.cc

 // Copyright 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 <limits>
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/process/process_metrics.h"
 
@@ skipping 56 matching lines @@
   sample_count_++;
 }
 
 void ProcessMetricsHistory::EndOfCycle() {
   RunPerformanceTriggers();
   ResetCounters();
 }
 
 void ProcessMetricsHistory::RunPerformanceTriggers() {
   // As an initial step, we only care about browser processes.
-  if (process_type_ != content::PROCESS_TYPE_BROWSER)
+  if (process_type_ != content::PROCESS_TYPE_BROWSER || sample_count_ == 0)
     return;
 
+  UMA_HISTOGRAM_CUSTOM_COUNTS("PerformanceMonitor.AverageCPU.BrowserProcess",
+                              accumulated_cpu_usage_ / sample_count_,
+                              0, 1200, 20);

[Alexei Svitkine (slow), 2013/10/18 21:39:12]

Can you explain why you're using these parameters? What's 1200?

[oystein (OOO til 10th of July), 2013/10/21 14:38:45]

1200 is the maximum value we'd see on a 12-core non-Windows system. It's
reasonably arbitrary, if anyone has any better suggestions.

[Alexei Svitkine (slow), 2013/10/21 14:53:46]

I'd suggest picking a value that's more future-proof (the current maximum of
today is not the maximum of tomorrow).

Alternatively, scale non-Windows values to 100% as the max and use
UMA_HISTOGRAM_PERCENTAGE() (which will also give you distinct buckets for all
values rather than exponential bucket ranges like UMA_HISTOGRAM_CUSTOM_COUNTS
will produce).

[oystein (OOO til 10th of July), 2013/10/21 15:15:05]

Sure, I'll bounce it to 6400 or something. 

[Alexei Svitkine (slow), 2013/10/21 15:27:31]

All right. And you're okay with the exponential bucket ranges provided by this
macro?

+
   // If CPU usage has consistently been above our threshold,
   // we *may* have an issue.
   if (min_cpu_usage_ > kHighCPUUtilizationThreshold)
     UMA_HISTOGRAM_BOOLEAN("PerformanceMonitor.HighCPU.BrowserProcess", true);
 }
 
 }  // namespace performance_monitor