cc:: Add RenderingStatsInstrumentation to manage collection of RenderingStats

cc/test/fake_layer_tree_host_impl.cc

Index: cc/test/fake_layer_tree_host_impl.cc
diff --git a/cc/test/fake_layer_tree_host_impl.cc b/cc/test/fake_layer_tree_host_impl.cc
index 334ff9fa66e921175a0bc7b4d25c7fc106c67a37..c1c3ddfc9e33e7d1f88adbf20f7b4d5aeccfee0e 100644
--- a/cc/test/fake_layer_tree_host_impl.cc
+++ b/cc/test/fake_layer_tree_host_impl.cc
@@ -7,7 +7,12 @@
 namespace cc {
 
 FakeLayerTreeHostImpl::FakeLayerTreeHostImpl(Proxy* proxy)
-    : LayerTreeHostImpl(LayerTreeSettings(), &client_, proxy) {
+    : rendering_stats_instrumentation_(RenderingStatsInstrumentation::Create()),

[tapted, 2013/03/21 02:27:22]

sheriff drive-by: I'm .. pretty sure the compiler will reorder these so the base
class will be initialized first, with an invalid pointer. 

+      LayerTreeHostImpl(LayerTreeSettings(),
+                        &client_,
+                        proxy,
+                        rendering_stats_instrumentation_.get())
+{
   // Explicitly clear all debug settings.
   SetDebugState(LayerTreeDebugState());
 }
@@ -15,7 +20,12 @@ FakeLayerTreeHostImpl::FakeLayerTreeHostImpl(Proxy* proxy)
 FakeLayerTreeHostImpl::FakeLayerTreeHostImpl(
     const LayerTreeSettings& settings,
     Proxy* proxy)
-    : LayerTreeHostImpl(settings, &client_, proxy) {
+    : rendering_stats_instrumentation_(RenderingStatsInstrumentation::Create()),
+      LayerTreeHostImpl(settings,
+                        &client_,
+                        proxy,
+                        rendering_stats_instrumentation_.get())
+{
   // Explicitly clear all debug settings.
   SetDebugState(LayerTreeDebugState());
 }