Move MaxScrollOffset to property_trees

cc/layers/layer.h

 // Copyright 2010 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_LAYERS_LAYER_H_
 #define CC_LAYERS_LAYER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 111 matching lines @@
   SkColor background_color() const { return background_color_; }
   // If contents_opaque(), return an opaque color else return a
   // non-opaque color.  Tries to return background_color(), if possible.
   SkColor SafeOpaqueBackgroundColor() const;
 
   // A layer's bounds are in logical, non-page-scaled pixels (however, the
   // root layer's bounds are in physical pixels).
   void SetBounds(const gfx::Size& bounds);
   gfx::Size bounds() const { return bounds_; }
 
+  // This function is for property tree builder

[ajuma, 2016/02/09 16:27:13]

Maybe point out this is because of templating? (Or, even better, make property
tree builder not depend on this, see comment there.)

+  gfx::Vector2dF bounds_delta() const { return gfx::Vector2dF(); }
+
   void SetMasksToBounds(bool masks_to_bounds);
   bool masks_to_bounds() const { return masks_to_bounds_; }
 
   void SetMaskLayer(Layer* mask_layer);
   Layer* mask_layer() { return mask_layer_.get(); }
   const Layer* mask_layer() const { return mask_layer_.get(); }
 
   virtual void SetNeedsDisplayRect(const gfx::Rect& dirty_rect);
   void SetNeedsDisplay() { SetNeedsDisplayRect(gfx::Rect(bounds())); }
 
@@ skipping 122 matching lines @@
   void SetScrollOffset(const gfx::ScrollOffset& scroll_offset);
   void SetScrollCompensationAdjustment(
       const gfx::Vector2dF& scroll_compensation_adjustment);
   gfx::Vector2dF ScrollCompensationAdjustment() const;
 
   gfx::ScrollOffset scroll_offset() const { return scroll_offset_; }
   void SetScrollOffsetFromImplSide(const gfx::ScrollOffset& scroll_offset);
 
   void SetScrollClipLayerId(int clip_layer_id);
   bool scrollable() const { return scroll_clip_layer_id_ != INVALID_ID; }
+  Layer* scroll_clip_layer() const;
+  int scroll_clip_layer_id() const { return scroll_clip_layer_id_; }

[ajuma, 2016/02/09 16:27:13]

It looks like this isn't called anywhere. Is it needed?

 
   void SetUserScrollable(bool horizontal, bool vertical);
   bool user_scrollable_horizontal() const {
     return user_scrollable_horizontal_;
   }
   bool user_scrollable_vertical() const { return user_scrollable_vertical_; }
 
   void AddMainThreadScrollingReasons(uint32_t main_thread_scrolling_reasons);
   void ClearMainThreadScrollingReasons();
   uint32_t main_thread_scrolling_reasons() const {
@@ skipping 506 matching lines @@
 
   std::vector<FrameTimingRequest> frame_timing_requests_;
   bool frame_timing_requests_dirty_;
 
   DISALLOW_COPY_AND_ASSIGN(Layer);
 };
 
 }  // namespace cc
 
 #endif  // CC_LAYERS_LAYER_H_