Record size of scroller that user scrolls

cc/trees/layer_tree_host_impl.h

Index: cc/trees/layer_tree_host_impl.h
diff --git a/cc/trees/layer_tree_host_impl.h b/cc/trees/layer_tree_host_impl.h
index f02a36c93d9f84b42c6df08717ce4986ccd47332..b4bcbe1884a2a951c1ee9d31bd0db409808700b9 100644
--- a/cc/trees/layer_tree_host_impl.h
+++ b/cc/trees/layer_tree_host_impl.h
@@ -605,6 +605,9 @@ class CC_EXPORT LayerTreeHostImpl
                         const base::Callback<void(bool)>& embedder_callback);
   std::vector<base::Closure> TakeCompletedImageDecodeCallbacks();
 
+  static const int maxScrollerSize = 200000;  // 400px * 500px
+  static const int bucketNum = 50;

[tdresser, 2017/03/30 15:19:06]

Prefix with "k"

https://google.github.io/styleguide/cppguide.html#Constant_Names

[weiliangc, 2017/03/30 19:19:36]

These two are just constants. Is it possible to put it in a header file
somewhere that can be included by both layer_tree_host_impl.cc and blink/core?
It doesn't seem like the constants need to be living on LayerTreeHostImpl class.

+
  protected:
   LayerTreeHostImpl(
       const LayerTreeSettings& settings,