Plumbing for rendering stats subscriber.

webkit/compositor_bindings/web_layer_tree_view_impl.h

Index: webkit/compositor_bindings/web_layer_tree_view_impl.h
diff --git a/webkit/compositor_bindings/web_layer_tree_view_impl.h b/webkit/compositor_bindings/web_layer_tree_view_impl.h
index ae2e45df2695a18ad177138453a4881d8710b7fb..23ee379bd7ff57aedeb05f39212c0c0b0acf3d67 100644
--- a/webkit/compositor_bindings/web_layer_tree_view_impl.h
+++ b/webkit/compositor_bindings/web_layer_tree_view_impl.h
@@ -49,6 +49,10 @@ public:
     virtual void finishAllRendering() OVERRIDE;
     virtual void setDeferCommits(bool deferCommits) OVERRIDE;
     virtual void renderingStats(WebRenderingStats&) const OVERRIDE;
+#ifdef RENDERING_STATS_SUBSCRIBER
+    virtual void startRecordingRenderingStats() const OVERRIDE { }

[danakj, 2012/11/02 21:15:19]

If you land the webkit side first, and don't make them pure virtual, then you
can probably get away without any ifdef magic?

[hartmanng, 2012/11/05 16:09:30]

Unless there's a strong consensus that this is the better way to do it, I'd
prefer to leave it as-is for now. I feel like this method "should" be pure
virtual, if nothing else it's consistent with the existing renderingStats()
method.

Plus I will have a 3rd patch regardless to plumb in the final javascript parts
(which can't land until after http://codereview.chromium.org/11198005 lands).

+    virtual void stopRecordingRenderingStats(WebRenderingStats& stats) const OVERRIDE { }
+#endif /* ifdef RENDERING_STATS_SUBSCRIBER */

[danakj, 2012/11/02 21:15:19]

nit: // RENDERING_STATS_SUBSCRIBER

(c++ style)

[hartmanng, 2012/11/05 16:09:30]

Done.

     virtual void setFontAtlas(SkBitmap, WebRect asciiToRectTable[128], int fontHeight) OVERRIDE;
     virtual void loseCompositorContext(int numTimes) OVERRIDE;