UI Resource Manager

cc/layers/scrollbar_layer.h

Index: cc/layers/scrollbar_layer.h
===================================================================
--- cc/layers/scrollbar_layer.h	(revision 210393)
+++ cc/layers/scrollbar_layer.h	(working copy)
@@ -10,13 +10,14 @@
 #include "cc/layers/contents_scaling_layer.h"
 #include "cc/layers/scrollbar_theme_painter.h"
 #include "cc/resources/layer_updater.h"
+#include "cc/resources/ui_resource_manager.h"
 
 namespace cc {
-class CachingBitmapContentLayerUpdater;
 class ResourceUpdateQueue;

[aelias_OOO_until_Jul13, 2013/07/10 23:07:22]

Also delete this.

[powei, 2013/07/11 23:54:44]

Done.

 class ScrollbarThemeComposite;
 
-class CC_EXPORT ScrollbarLayer : public ContentsScalingLayer {
+class CC_EXPORT ScrollbarLayer : public ContentsScalingLayer,
+  public UIResourceManagerClient {

[aelias_OOO_until_Jul13, 2013/07/10 23:07:22]

nit: wrong spacing

[powei, 2013/07/11 23:54:44]

Done.

  public:
   virtual scoped_ptr<LayerImpl> CreateLayerImpl(LayerTreeImpl* tree_impl)
       OVERRIDE;
@@ -33,8 +34,6 @@
   ScrollbarOrientation Orientation() const;
 
   // Layer interface
-  virtual void SetTexturePriorities(const PriorityCalculator& priority_calc)
-      OVERRIDE;
   virtual void Update(ResourceUpdateQueue* queue,

[aelias_OOO_until_Jul13, 2013/07/10 23:07:22]

This should be deleted.

[powei, 2013/07/11 23:54:44]

Isn't this function where the bitmaps are generated?  If not here, then where
should the texture updates go? 

[aelias_OOO_until_Jul13, 2013/07/12 00:52:27]

This function is tied to the legacy PrioritizedResourceManager system, that's
why we should delete it.

I believe the only reason to reraster scrollbars is when contents scale changes.
 So you could trigger raster at the end of CalculateContentsScale (after
checking that resulted in a change from the previous value).

                       const OcclusionTracker* occlusion) OVERRIDE;
   virtual void SetLayerTreeHost(LayerTreeHost* host) OVERRIDE;
@@ -49,17 +48,14 @@
 
   virtual ScrollbarLayer* ToScrollbarLayer() OVERRIDE;
 
+  virtual void UIResourceReady(UIResourceId id) OVERRIDE;

[aelias_OOO_until_Jul13, 2013/07/10 23:07:22]

No need for this here, it should have an empty implementation in the base class.

[powei, 2013/07/11 23:54:44]

Done.

+
  protected:
   ScrollbarLayer(scoped_ptr<Scrollbar> scrollbar,
                  int scroll_layer_id);
   virtual ~ScrollbarLayer();
 
  private:
-  void UpdatePart(CachingBitmapContentLayerUpdater* painter,
-                  LayerUpdater::Resource* resource,
-                  gfx::Rect rect,
-                  ResourceUpdateQueue* queue);
-  void CreateUpdaterIfNeeded();
   gfx::Rect ScrollbarLayerRectToContentRect(gfx::Rect layer_rect) const;
   gfx::Rect OriginThumbRect() const;
 
@@ -75,17 +71,14 @@
   gfx::Rect track_rect_;
   int scroll_layer_id_;
 
-  unsigned texture_format_;
-
   gfx::RectF dirty_rect_;
 
-  scoped_refptr<CachingBitmapContentLayerUpdater> track_updater_;
-  scoped_refptr<CachingBitmapContentLayerUpdater> thumb_updater_;
+  UIResourceId track_ui_resource_id_;
+  UIResourceId thumb_ui_resource_id_;
+  scoped_refptr<UIResourceBitmap> track_bitmap_;
+  scoped_refptr<UIResourceBitmap> thumb_bitmap_;
+  bool skip_update_;

[aelias_OOO_until_Jul13, 2013/07/10 23:07:22]

It doesn't look like this variable does anything useful, please delete it.

[powei, 2013/07/11 23:54:44]

Done.

 
-  // All the parts of the scrollbar except the thumb
-  scoped_ptr<LayerUpdater::Resource> track_;
-  scoped_ptr<LayerUpdater::Resource> thumb_;
-
   DISALLOW_COPY_AND_ASSIGN(ScrollbarLayer);
 };