[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to views::ScrollView

[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on the views::ScrollView scroll track
[landed] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
[landed] http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2197503002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

[tapted, 2016-07-28 01:48:28]

Description was changed from

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
==========

to

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
==========

[tapted, 2016-07-28 01:49:31]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-28 01:49:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 01:49:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-28 01:49:51]

Dry run: Your CL was about to rely on recently removed CQ feature(s):
 * master.tryserver.blink builder linux_blink_rel was renamed to
linux_precise_blink_rel (http://crbug.com/590036)

[tapted, 2016-07-28 01:50:39]

Description was changed from

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
==========

to

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
==========

[tapted, 2016-07-28 01:51:12]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-28 01:51:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 01:53:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-28 01:53:27]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[tapted, 2016-07-28 03:20:36]

Description was changed from

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
==========

to

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[tapted, 2016-07-28 04:28:24]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-28 04:28:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 04:35:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-28 04:35:33]

Dry run: Try jobs failed on following builders:
  linux_precise_blink_rel on master.tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_precise_blink_rel/...)

[tapted, 2016-07-29 08:13:00]

Description was changed from

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on the
views::ScrollView scroll track
http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[tapted, 2016-07-29 08:38:44]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-29 08:38:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-29 08:48:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-29 08:48:25]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tapted, 2016-08-02 13:15:09]

Description was changed from

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on the
views::ScrollView scroll track
http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[published] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[sky, 2016-08-02 17:29:44]

sky@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2016-08-02 17:29:45]

https://codereview.chromium.org/2189583004/diff/80001/ui/events/event_constan...
File ui/events/event_constants.h (right):

https://codereview.chromium.org/2189583004/diff/80001/ui/events/event_constan...
ui/events/event_constants.h:169: EM_PHASE_MAY_BEGIN = 0x1,
Use 1 << 0, 1 << 1 .... to be consistent with EventFlags above.

https://codereview.chromium.org/2189583004/diff/80001/ui/views/controls/scrol...
File ui/views/controls/scroll_view.cc (right):

https://codereview.chromium.org/2189583004/diff/80001/ui/views/controls/scrol...
ui/views/controls/scroll_view.cc:198:
contents_container_->SetPaintToLayer(true);
Would it be possible to to turn on paint to layer and setscrollable for
contents_ directly? Otherwise if contents_ already paints to a layer we are
unnecessarily creating layers.

[tapted, 2016-08-03 00:14:44]

Description was changed from

==========
Adding Elastic+Momentum+Layered scrolling to views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[published] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[published] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[tapted, 2016-08-03 00:18:58]

https://codereview.chromium.org/2189583004/diff/80001/ui/events/event_constan...
File ui/events/event_constants.h (right):

https://codereview.chromium.org/2189583004/diff/80001/ui/events/event_constan...
ui/events/event_constants.h:169: EM_PHASE_MAY_BEGIN = 0x1,
On 2016/08/02 17:29:45, sky wrote:
> Use 1 << 0, 1 << 1 .... to be consistent with EventFlags above.

I'll update http://crrev.com/2193153002 with this and get things rebased.

https://codereview.chromium.org/2189583004/diff/80001/ui/views/controls/scrol...
File ui/views/controls/scroll_view.cc (right):

https://codereview.chromium.org/2189583004/diff/80001/ui/views/controls/scrol...
ui/views/controls/scroll_view.cc:198:
contents_container_->SetPaintToLayer(true);
On 2016/08/02 17:29:45, sky wrote:
> Would it be possible to to turn on paint to layer and setscrollable for
> contents_ directly? Otherwise if contents_ already paints to a layer we are
> unnecessarily creating layers.

One issue was that there's no guarantee that |contents_| will fill the viewport
after layout. (E.g. viewport: 100x100, content: 50x200 => but
contents_container_ will be 100*x200). LayerTreeHostImpl does hit-testing of the
scroll events to find the layer to scroll -- if the layer doesn't extend the
full width of the viewport then hit tests fail.

Other reasons
 - it might be weird for the on-scroll callback to be directed to ScrollView for
a layer it didn't create
 - ScrollView may need other guarantees about the layer properties - e.g. it
should be opaque, and it manipulates a bunch of properties in
ui::Compositor::SetScrollable
 - the separate container filling the viewport gives the correct effect during
overscroll (i.e. whole viewport gets elasticity, and a background may become
visible during overscroll [although currently it just blends white with white]).


I don't think any of these are blockers for calling SetScrollable on |contents|
itself, but I think it would be tricky to coordinate the layout requirement of
|contents|. That is, I don't think we can avoid the requirement that the
scrolling layer is at least as wide and at least as tall as the viewport - and
that would mess up the existing layout logic.


Perhaps the use-case for contents requiring its own layer could be obsolete, if
the scroll view has it instead? (e.g. we could DCHECK that the contents doesn't
have a layer).


(*actually the container width would be 100 minus a bit for the vertical
scroller, if present.)

[sky, 2016-08-03 15:25:17]

On Tue, Aug 2, 2016 at 5:18 PM,  <tapted@chromium.org> wrote:
> Reviewers: sky
> CL: https://codereview.chromium.org/2189583004/
>
>
>
https://codereview.chromium.org/2189583004/diff/80001/ui/events/event_constan...
> File ui/events/event_constants.h (right):
>
>
https://codereview.chromium.org/2189583004/diff/80001/ui/events/event_constan...
> ui/events/event_constants.h:169: EM_PHASE_MAY_BEGIN = 0x1,
> On 2016/08/02 17:29:45, sky wrote:
>> Use 1 << 0, 1 << 1 .... to be consistent with EventFlags above.
>
> I'll update http://crrev.com/2193153002 with this and get things
> rebased.
>
>
https://codereview.chromium.org/2189583004/diff/80001/ui/views/controls/scrol...
> File ui/views/controls/scroll_view.cc (right):
>
>
https://codereview.chromium.org/2189583004/diff/80001/ui/views/controls/scrol...
> ui/views/controls/scroll_view.cc:198:
> contents_container_->SetPaintToLayer(true);
> On 2016/08/02 17:29:45, sky wrote:
>> Would it be possible to to turn on paint to layer and setscrollable
> for
>> contents_ directly? Otherwise if contents_ already paints to a layer
> we are
>> unnecessarily creating layers.
>
> One issue was that there's no guarantee that |contents_| will fill the
> viewport after layout. (E.g. viewport: 100x100, content: 50x200 => but
> contents_container_ will be 100*x200). LayerTreeHostImpl does
> hit-testing of the scroll events to find the layer to scroll -- if the
> layer doesn't extend the full width of the viewport then hit tests fail.
>
> Other reasons
> - it might be weird for the on-scroll callback to be directed to
> ScrollView for a layer it didn't create
> - ScrollView may need other guarantees about the layer properties -
> e.g. it should be opaque, and it manipulates a bunch of properties in
> ui::Compositor::SetScrollable
> - the separate container filling the viewport gives the correct effect
> during overscroll (i.e. whole viewport gets elasticity, and a background
> may become visible during overscroll [although currently it just blends
> white with white]).
>
>
> I don't think any of these are blockers for calling SetScrollable on
> |contents| itself, but I think it would be tricky to coordinate the
> layout requirement of |contents|. That is, I don't think we can avoid
> the requirement that the scrolling layer is at least as wide and at
> least as tall as the viewport - and that would mess up the existing
> layout logic.
>
>
> Perhaps the use-case for contents requiring its own layer could be
> obsolete, if the scroll view has it instead? (e.g. we could DCHECK that
> the contents doesn't have a layer).

If making scrollview have the layer means we don't need an additional
view with a layer that seems good to me.

  -Scott

>
>
> (*actually the container width would be 100 minus a bit for the vertical
> scroller, if present.)
>
> Description:
> [not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
> views::ScrollView
>
> Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
> many patchsets
>
> Current split looks like:
>
> [landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking
> on
> the views::ScrollView scroll track
> [published] http://crrev.com/2188133002 Scroll with Layers in
> views::ScrollView
> http://crrev.com/2193153002 MacViews: Send scrollWheel events as
> ui::ET_SCROLL
> http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
> http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView
>
> BUG=615948, 594907, 605131, 355659, 546520, 585766
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Base URL: https://chromium.googlesource.com/chromium/src.git@master
>
> Affected files (+1581, -354 lines):
> M cc/blink/web_layer_impl.cc
> M cc/input/input_handler.h
> M cc/input/scroll_elasticity_helper.h
> M cc/input/scroll_elasticity_helper.cc
> M cc/input/scroll_state.h
> M cc/layers/layer.h
> M cc/layers/layer.cc
> M cc/layers/layer_impl.h
> M cc/layers/layer_impl.cc
> M cc/layers/layer_unittest.cc
> M cc/proto/begin_main_frame_and_commit_state.proto
> M cc/proto/layer.proto
> M cc/proto/property_tree.proto
> M cc/test/layer_tree_json_parser.cc
> M cc/test/layer_tree_test.cc
> M cc/test/test_layer_tree_host_base.cc
> M cc/trees/draw_property_utils.cc
> M cc/trees/layer_tree_host.cc
> M cc/trees/layer_tree_host_common.h
> M cc/trees/layer_tree_host_common.cc
> M cc/trees/layer_tree_host_common_unittest.cc
> M cc/trees/layer_tree_host_impl.h
> M cc/trees/layer_tree_host_impl.cc
> M cc/trees/layer_tree_host_unittest.cc
> M cc/trees/layer_tree_impl.h
> M cc/trees/layer_tree_impl.cc
> M cc/trees/property_tree.h
> M cc/trees/property_tree.cc
> M cc/trees/property_tree_builder.cc
> M cc/trees/property_tree_unittest.cc
> M cc/trees/single_thread_proxy.cc
> M cc/trees/transform_node.h
> M cc/trees/transform_node.cc
> M cc/trees/tree_synchronizer_unittest.cc
> M ui/compositor/BUILD.gn
> M ui/compositor/compositor.h
> M ui/compositor/compositor.cc
> M ui/compositor/compositor.gyp
> M ui/compositor/layer.h
> M ui/compositor/layer.cc
> A + ui/compositor/overscroll/DEPS
> A ui/compositor/overscroll/ui_input_handler.h
> A ui/compositor/overscroll/ui_input_handler.cc
> A ui/compositor/overscroll/ui_scroll_input_manager.h
> A ui/compositor/overscroll/ui_scroll_input_manager.cc
> M ui/events/blink/input_handler_proxy.cc
> M ui/events/blink/input_handler_proxy_unittest.cc
> M ui/events/blink/input_scroll_elasticity_controller.h
> M ui/events/blink/input_scroll_elasticity_controller.cc
> M ui/events/blink/input_scroll_elasticity_controller_unittest.cc
> M ui/events/cocoa/events_mac.mm
> M ui/events/event.h
> M ui/events/event.cc
> M ui/events/event_constants.h
> M ui/events/event_utils.h
> M ui/events/event_utils.cc
> M ui/events/events_default.cc
> M ui/events/events_stub.cc
> M ui/events/win/events_win.cc
> M ui/events/x/events_x.cc
> M ui/views/cocoa/bridged_content_view.mm
> M ui/views/controls/scroll_view.h
> M ui/views/controls/scroll_view.cc
> M ui/views/controls/scroll_view_unittest.cc
> M ui/views/controls/scrollbar/base_scroll_bar.h
> M ui/views/controls/scrollbar/cocoa_scroll_bar.mm
> M ui/views/controls/scrollbar/native_scroll_bar.h
> M ui/views/controls/table/table_view.cc
> M ui/views/examples/scroll_view_example.h
> M ui/views/examples/scroll_view_example.cc
> M ui/views/repeat_controller.h
> M ui/views/view.h
> M ui/views/view.cc
>
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[tapted, 2016-08-04 08:45:54]

tapted@chromium.org changed reviewers:
- sky@chromium.org

[tapted, 2016-08-08 07:47:29]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-08 07:47:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-08 07:57:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-08 07:57:20]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tapted, 2016-08-15 02:39:07]

Description was changed from

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[published] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[landed] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[tapted, 2016-08-15 02:40:25]

Description was changed from

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[landed] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2193153002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[landed] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2197503002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[tapted, 2016-09-27 04:26:14]

Description was changed from

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[landed] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
http://crrev.com/2193153002 MacViews: Send scrollWheel events as ui::ET_SCROLL
http://crrev.com/2197503002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[not for review - epic CL] Adding Elastic+Momentum+Layered scrolling to
views::ScrollView

Old CL: https://codereview.chromium.org/1680613002/ - accumulated too
many patchsets

Current split looks like:

[landed] http://crrev.com/2192443003 MacViews: Fix scrolling when clicking on
the views::ScrollView scroll track
[landed] http://crrev.com/2188133002 Scroll with Layers in views::ScrollView
[landed] http://crrev.com/2193153002 MacViews: Send scrollWheel events as
ui::ET_SCROLL
http://crrev.com/2197503002 Route Scroll events through a cc::InputHandler
http://crrev.com/2194833002 Overscroll and Elasticity for views::ScrollView

BUG=615948, 594907, 605131, 355659, 546520, 585766
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[tapted, 2016-10-07 08:19:17]

Patchset #9 (id:160001) has been deleted