Refactor cc scrollbar layers to separate solid-color vs desktop.

cc/layers/scrollbar_layer_impl.h

Index: cc/layers/scrollbar_layer_impl.h
diff --git a/cc/layers/scrollbar_layer_impl.h b/cc/layers/scrollbar_layer_impl.h
index f61b6608b51107ca83521e60bc822852bf736a5e..ba05185fc0609521aaca204d259f2863f99bd78e 100644
--- a/cc/layers/scrollbar_layer_impl.h
+++ b/cc/layers/scrollbar_layer_impl.h
@@ -7,14 +7,14 @@
 
 #include "cc/base/cc_export.h"
 #include "cc/input/scrollbar.h"
-#include "cc/layers/layer_impl.h"
+#include "cc/layers/scrollbar_layer_impl_base.h"
 
 namespace cc {
 
 class LayerTreeImpl;
 class ScrollView;
 
-class CC_EXPORT ScrollbarLayerImpl : public LayerImpl {
+class CC_EXPORT ScrollbarLayerImpl : public ScrollbarLayerImplBase {
  public:
   static scoped_ptr<ScrollbarLayerImpl> Create(
       LayerTreeImpl* tree_impl,
@@ -23,7 +23,6 @@ class CC_EXPORT ScrollbarLayerImpl : public LayerImpl {
   virtual ~ScrollbarLayerImpl();
 
   // LayerImpl implementation.
-  virtual ScrollbarLayerImpl* ToScrollbarLayer() OVERRIDE;
   virtual scoped_ptr<LayerImpl> CreateLayerImpl(LayerTreeImpl* tree_impl)
       OVERRIDE;
   virtual void PushPropertiesTo(LayerImpl* layer) OVERRIDE;
@@ -35,13 +34,6 @@ class CC_EXPORT ScrollbarLayerImpl : public LayerImpl {
 
   virtual void DidLoseOutputSurface() OVERRIDE;
 
-  int scroll_layer_id() const { return scroll_layer_id_; }
-  void set_scroll_layer_id(int id) { scroll_layer_id_ = id; }
-
-  ScrollbarOrientation Orientation() const;
-  float CurrentPos() const;
-  int Maximum() const;
-
   void set_thumb_thickness(int thumb_thickness) {
     thumb_thickness_ = thumb_thickness;
   }
@@ -55,43 +47,29 @@ class CC_EXPORT ScrollbarLayerImpl : public LayerImpl {
   void set_track_length(int track_length) {
     track_length_ = track_length;
   }
-  void set_vertical_adjust(float vertical_adjust) {
-    vertical_adjust_ = vertical_adjust;
-  }
   void set_track_resource_id(ResourceProvider::ResourceId id) {
     track_resource_id_ = id;
   }
   void set_thumb_resource_id(ResourceProvider::ResourceId id) {
     thumb_resource_id_ = id;
   }
-  void set_visible_to_total_length_ratio(float ratio) {
-    visible_to_total_length_ratio_ = ratio;
-  }
-  void set_is_overlay_scrollbar(bool is_overlay_scrollbar) {
-    is_overlay_scrollbar_ = is_overlay_scrollbar;
-  }
-  bool is_overlay_scrollbar() const { return is_overlay_scrollbar_; }
-
-  void SetCurrentPos(float current_pos) { current_pos_ = current_pos; }
-  void SetMaximum(int maximum) { maximum_ = maximum; }
-
-  gfx::Rect ComputeThumbQuadRect() const;
 
  protected:
   ScrollbarLayerImpl(LayerTreeImpl* tree_impl,
                      int id,
                      ScrollbarOrientation orientation);
 
+  virtual int ThumbThickness() const OVERRIDE;

[enne (OOO), 2013/08/14 21:01:09]

Comment about what class's functions these are overriding?

[wjmaclean, 2013/08/15 20:09:40]

Done.

+  virtual int ThumbLength() const OVERRIDE;
+  virtual float TrackLength() const OVERRIDE;
+  virtual int TrackStart() const OVERRIDE;
+
  private:
   virtual const char* LayerTypeAsString() const OVERRIDE;
 
-  gfx::Rect ScrollbarLayerRectToContentRect(gfx::RectF layer_rect) const;
-
   ResourceProvider::ResourceId track_resource_id_;
   ResourceProvider::ResourceId thumb_resource_id_;
 
-  float current_pos_;
-  int maximum_;
   int thumb_thickness_;
   int thumb_length_;
   int track_start_;
@@ -106,8 +84,6 @@ class CC_EXPORT ScrollbarLayerImpl : public LayerImpl {
 
   int scroll_layer_id_;
 
-  bool is_overlay_scrollbar_;
-
   DISALLOW_COPY_AND_ASSIGN(ScrollbarLayerImpl);
 };