UMA metric for LayerUpdateTimes

cc/trees/layer_tree_host_in_process.cc

Index: cc/trees/layer_tree_host_in_process.cc
diff --git a/cc/trees/layer_tree_host_in_process.cc b/cc/trees/layer_tree_host_in_process.cc
index 8a9d969f3a6e84dd2ccfa29311a4a836ea8cfa63..a394067e825091318b752999e1830d62f51c1021 100644
--- a/cc/trees/layer_tree_host_in_process.cc
+++ b/cc/trees/layer_tree_host_in_process.cc
@@ -23,9 +23,12 @@
 #include "base/single_thread_task_runner.h"
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
+#include "base/strings/stringprintf.h"
 #include "base/threading/thread_task_runner_handle.h"
+#include "base/timer/elapsed_timer.h"
 #include "base/trace_event/trace_event.h"
 #include "base/trace_event/trace_event_argument.h"
+#include "cc/base/histograms.h"
 #include "cc/base/math_util.h"
 #include "cc/blimp/client_picture_cache.h"
 #include "cc/blimp/engine_picture_cache.h"
@@ -582,12 +585,39 @@ void LayerTreeHostInProcess::Composite(base::TimeTicks frame_begin_time) {
   proxy->CompositeImmediately(frame_begin_time);
 }
 
+static int GetLayersUpdateTimeHistogramBucket(size_t numLayers) {
+  // We uses the following exponential (ratio 2) bucketization:
+  // [0, 10), [10, 30), [30, 70), [70, 150), [150, infinity)
+  if (numLayers < 10)
+    return 0;
+  else if (numLayers >= 10 && numLayers < 30)

[flackr, 2016/12/08 22:35:08]

nit: >= condition in each of these is unnecessary given that we haven't returned
yet.

[majidvp, 2016/12/19 20:39:33]

Done.

+    return 1;
+  else if (numLayers >= 30 && numLayers < 70)
+    return 2;
+  else if (numLayers >= 70 && numLayers < 150)
+    return 3;
+  else

[flackr, 2016/12/08 22:35:08]

per the style guide, no else (or else if) after return.

[majidvp, 2016/12/19 20:39:33]

Done.

+    return 4;
+}
+
 bool LayerTreeHostInProcess::UpdateLayers() {
   if (!layer_tree_->root_layer())
     return false;
   DCHECK(!layer_tree_->root_layer()->parent());
+  base::ElapsedTimer timer;
+
   bool result = DoUpdateLayers(layer_tree_->root_layer());
   micro_benchmark_controller_.DidUpdateLayers();
+
+  if (const char* client_name = GetClientNameForMetrics()) {
+    std::string histogram_name = base::StringPrintf(
+        "Compositing.%s.LayersUpdateTime.%d", client_name,
+        GetLayersUpdateTimeHistogramBucket(layer_tree_->NumLayers()));
+    base::Histogram::FactoryGet(histogram_name, 0, 10000000, 50,
+                                base::HistogramBase::kUmaTargetedHistogramFlag)
+        ->Add(timer.Elapsed().InMicroseconds());
+  }
+
   return result || next_commit_forces_redraw_;
 }