cc: handling debug settings in new LayerTreeDebugState structure

cc/heads_up_display_layer.cc

Index: cc/heads_up_display_layer.cc
diff --git a/cc/heads_up_display_layer.cc b/cc/heads_up_display_layer.cc
index 374e650e777ff5f346219341b07535ec9b93fd95..c68a6e46e0564d7b3a87a0abe333d61e6d3c391a 100644
--- a/cc/heads_up_display_layer.cc
+++ b/cc/heads_up_display_layer.cc
@@ -17,7 +17,6 @@ scoped_refptr<HeadsUpDisplayLayer> HeadsUpDisplayLayer::create()
 
 HeadsUpDisplayLayer::HeadsUpDisplayLayer()
     : Layer()
-    , m_showFPSCounter(false)
 {
     setBounds(gfx::Size(256, 128));
 }
@@ -28,14 +27,14 @@ HeadsUpDisplayLayer::~HeadsUpDisplayLayer()
 
 void HeadsUpDisplayLayer::update(ResourceUpdateQueue&, const OcclusionTracker*, RenderingStats&)
 {
-    const LayerTreeSettings& settings = layerTreeHost()->settings();
+    const LayerTreeDebugState& debugState = layerTreeHost()->debugState();
     int maxTextureSize = layerTreeHost()->rendererCapabilities().maxTextureSize;
 
     gfx::Size bounds;
     gfx::Transform matrix;
     matrix.MakeIdentity();
 
-    if (settings.showPlatformLayerTree || settings.showDebugRects()) {
+    if (debugState.showPlatformLayerTree || debugState.showHudRects()) {
         int width = std::min(maxTextureSize, layerTreeHost()->deviceViewportSize().width());
         int height = std::min(maxTextureSize, layerTreeHost()->deviceViewportSize().height());
         bounds = gfx::Size(width, height);
@@ -59,12 +58,6 @@ void HeadsUpDisplayLayer::setFontAtlas(scoped_ptr<FontAtlas> fontAtlas)
     setNeedsCommit();
 }
 
-void HeadsUpDisplayLayer::setShowFPSCounter(bool show)
-{
-    m_showFPSCounter = show;
-    setNeedsCommit();
-}
-
 scoped_ptr<LayerImpl> HeadsUpDisplayLayer::createLayerImpl()
 {
     return HeadsUpDisplayLayerImpl::create(m_layerId).PassAs<LayerImpl>();
@@ -74,11 +67,10 @@ void HeadsUpDisplayLayer::pushPropertiesTo(LayerImpl* layerImpl)
 {
     Layer::pushPropertiesTo(layerImpl);
 
-    HeadsUpDisplayLayerImpl* hudLayerImpl = static_cast<HeadsUpDisplayLayerImpl*>(layerImpl);
-    hudLayerImpl->setShowFPSCounter(m_showFPSCounter);
-
-    if (m_fontAtlas.get())
+    if (m_fontAtlas.get()) {

[danakj, 2012/11/27 20:23:15]

instead of nesting here, just early-out when (!m_fontAtlas)

.get() is redundant also :)

+        HeadsUpDisplayLayerImpl* hudLayerImpl = static_cast<HeadsUpDisplayLayerImpl*>(layerImpl);
         hudLayerImpl->setFontAtlas(m_fontAtlas.Pass());
+    }
 }
 
 }  // namespace cc