toggle FPS counter in compositor

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 f9b9e11286c116558652dfdec06261a0a7038be3..7c400419a19378179b2940b69a34223c1bdaa97f 100644
--- a/webkit/compositor_bindings/web_layer_tree_view_impl.h
+++ b/webkit/compositor_bindings/web_layer_tree_view_impl.h
@@ -47,6 +47,11 @@ public:
     virtual bool compositeAndReadback(void *pixels, const WebRect&) OVERRIDE;
     virtual void finishAllRendering() OVERRIDE;
     virtual void renderingStats(WebRenderingStats&) const OVERRIDE;
+
+    // NOTE: not marked OVERRIDE to avoid dependency till associated WebKit patch is landed: https://bugs.webkit.org/show_bug.cgi?id=99660

[danakj, 2012/10/23 15:19:53]

Don't worry about adding OVERRIDE to new methods here. The current OVERRIDEs are
just there because they were already there when we moved stuff here.

+    virtual void setShowFPSCounter(bool show);
+    virtual bool hasFontAtlas();
+
     virtual void setFontAtlas(SkBitmap, WebRect asciiToRectTable[128], int fontHeight) OVERRIDE;
     virtual void loseCompositorContext(int numTimes) OVERRIDE;
 
@@ -68,6 +73,7 @@ public:
 private:
     WebLayerTreeViewClient* m_client;
     scoped_ptr<cc::CCLayerTreeHost> m_layerTreeHost;
+    bool m_hasFontAtlas;

[danakj, 2012/10/23 15:19:53]

How about moving this piece of state into CCLayerTreeHost so we can keep this
impl class as just a wrapper? It can get/set this value from the
CCLayerTreeHost.

Or, even better, into the hud layer? Then
CCLayerTreeHost::hasFontAtlas() { return m_hudLayer &&
m_hudLayer->hasFontAtlast(); }

 };
 
 } // namespace WebKit