cc: switch to new WebLayerTreeViewClient API for requesting font atlas

cc/heads_up_display_layer.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_HEADS_UP_DISPLAY_LAYER_H_
 #define CC_HEADS_UP_DISPLAY_LAYER_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "cc/font_atlas.h"
 #include "cc/layer.h"
 
 namespace cc {
 
 class HeadsUpDisplayLayer : public Layer {
 public:
     static scoped_refptr<HeadsUpDisplayLayer> create();
 
     virtual void update(ResourceUpdateQueue&, const OcclusionTracker*, RenderingStats&) OVERRIDE;
     virtual bool drawsContent() const OVERRIDE;
 
     void setFontAtlas(scoped_ptr<FontAtlas>);
     void setShowFPSCounter(bool);
 
     virtual scoped_ptr<LayerImpl> createLayerImpl() OVERRIDE;
     virtual void pushPropertiesTo(LayerImpl*) OVERRIDE;
 
+    bool hasFontAtlas() const { return m_hasFontAtlas; }
+
 protected:
     HeadsUpDisplayLayer();
 
 private:
     virtual ~HeadsUpDisplayLayer();
 
     scoped_ptr<FontAtlas> m_fontAtlas;
     bool m_showFPSCounter;
+
+    bool m_hasFontAtlas;

[jamesr, 2012/11/27 21:07:44]

why do we need a bool? Can't we just null-check m_fontAtlas?

[danakj, 2012/11/27 21:26:14]

This seems needed to remember we gave a font atlas to the impl thread.

[egraether, 2012/11/27 21:49:22]

Yeah, it's there because as soon as the fontAtlas is passed to the HudLayerImpl
you can't check it anymore.

This check will probably fall off, as soon as I start fixing the hudLayer
creation/destruction.

 };
 
 }  // namespace cc
 
 #endif  // CC_HEADS_UP_DISPLAY_LAYER_H_