Store non-fast scrollable regions on ScrollNode

cc/trees/property_tree_builder.cc

Index: cc/trees/property_tree_builder.cc
diff --git a/cc/trees/property_tree_builder.cc b/cc/trees/property_tree_builder.cc
index af02c5f5c62a58350abcf55edf66c54eb589d4ce..d9d48605fd3d3cdb15447f7654bb28e804ddd6a9 100644
--- a/cc/trees/property_tree_builder.cc
+++ b/cc/trees/property_tree_builder.cc
@@ -1163,8 +1163,7 @@ void AddScrollNodeIfNeeded(
     node.owning_layer_id = layer->id();
     node.scrollable = scrollable;
     node.main_thread_scrolling_reasons = main_thread_scrolling_reasons;
-    node.contains_non_fast_scrollable_region =
-        contains_non_fast_scrollable_region;
+    node.non_fast_scrollable_region = layer->non_fast_scrollable_region();

[ajuma, 2017/02/24 14:27:57]

Is LayerImpl::non_fast_scrollable_region_ now unused (except maybe in tests)?
Can we move it to LayerImplTestProperties?

[pdr., 2017/02/27 23:40:22]

This is used by debug_rect_history too. I started pulling on that thread but it
goes pretty deep.

[pdr., 2017/02/28 19:36:47]

Are you okay with not doing this in this patch?

[ajuma, 2017/02/28 19:38:59]

Yeah, leaving that for later sgtm.

     gfx::Size clip_bounds;
     if (layer->scroll_clip_layer()) {
       clip_bounds = layer->scroll_clip_layer()->bounds();