toggle FPS counter in compositor

webkit/compositor_bindings/WebLayerTreeViewImpl.h

Index: webkit/compositor_bindings/WebLayerTreeViewImpl.h
diff --git a/webkit/compositor_bindings/WebLayerTreeViewImpl.h b/webkit/compositor_bindings/WebLayerTreeViewImpl.h
index f9b9e11286c116558652dfdec06261a0a7038be3..d0c9d7e9aabe9eb21dad92315c15ee38091869b6 100644
--- a/webkit/compositor_bindings/WebLayerTreeViewImpl.h
+++ b/webkit/compositor_bindings/WebLayerTreeViewImpl.h
@@ -49,6 +49,8 @@ public:
     virtual void renderingStats(WebRenderingStats&) const OVERRIDE;
     virtual void setFontAtlas(SkBitmap, WebRect asciiToRectTable[128], int fontHeight) OVERRIDE;
     virtual void loseCompositorContext(int numTimes) OVERRIDE;
+    virtual void setShowFPSCounter(bool show) OVERRIDE;
+    virtual bool hasFontAtlas() OVERRIDE;

[nduca, 2012/10/20 19:49:41]

You're going to need to set up your patches to land one at a time. So, for
exmple, since you've written OVERRIDE here, you can't actually land this until
the code on the webkit side lands. And, your webkit-side code probably has this
set as pure virtual. Which means that if you land the webkit side without your
chrome changes, the chrome side will break because this file wont have these
changes.

I suggest you land the webkit side change first, ultimately, and not make these
methods pure virtual. Then you can land these after just as they are.

[nduca, 2012/10/20 19:49:41]

Also, put hasFontAtlas next to the setFontAtlas function. They are related.

 
     // cc::CCLayerTreeHostClient implementation.
     virtual void willBeginFrame() OVERRIDE;
@@ -68,6 +70,7 @@ public:
 private:
     WebLayerTreeViewClient* m_client;
     scoped_ptr<cc::CCLayerTreeHost> m_layerTreeHost;
+    bool m_hasFontAtlas;
 };
 
 } // namespace WebKit