Scale the deltaX/deltaY on the mouse wheel events to be consistent with the compositor implementati…

Scale the deltaX/deltaY on the mouse wheel events to be consistent with the compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441
Committed: https://crrev.com/aa27c866cdef570e9788f9a93a3a3d87aa36ebce
Cr-Commit-Position: refs/heads/master@{#381702}

[dtapuska, 2016-03-16 21:42:06]

Description was changed from

==========
Scale the deltaX/deltaY on the mouse wheel events to be consistent with the
compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441
==========

to

==========
Scale the deltaX/deltaY on the mouse wheel events to be consistent with the
compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441
==========

[dtapuska, 2016-03-16 21:42:06]

dtapuska@chromium.org changed reviewers:
+ bokan@chromium.org

[bokan, 2016-03-17 12:31:16]

lgtm

https://codereview.chromium.org/1812653002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/events/gesture-pinch-zoom-scroll-bubble.html
(right):

https://codereview.chromium.org/1812653002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/gesture-pinch-zoom-scroll-bubble.html:9:
eventSender.continuousMouseScrollBy(0, -2, false, true);
Why do we need the arguments on only this call?

[dtapuska, 2016-03-17 12:48:11]

https://codereview.chromium.org/1812653002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/events/gesture-pinch-zoom-scroll-bubble.html
(right):

https://codereview.chromium.org/1812653002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/gesture-pinch-zoom-scroll-bubble.html:9:
eventSender.continuousMouseScrollBy(0, -2, false, true);
On 2016/03/17 12:31:16, bokan wrote:
> Why do we need the arguments on only this call?

Because I accidentally left them in from debugging the true issue. Removed.

[dtapuska, 2016-03-17 12:48:19]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-03-17 12:48:19]

The patchset sent to the CQ was uploaded after l-g-t-m from bokan@chromium.org
Link to the patchset: https://codereview.chromium.org/1812653002/#ps40001
(title: "Remove unnecessary arguments from call")

[commit-bot: I haz the power, 2016-03-17 12:48:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1812653002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1812653002/40001

[commit-bot: I haz the power, 2016-03-17 13:59:34]

Description was changed from

==========
Scale the deltaX/deltaY on the mouse wheel events to be consistent with the
compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441
==========

to

==========
Scale the deltaX/deltaY on the mouse wheel events to be consistent with the
compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441
==========

[commit-bot: I haz the power, 2016-03-17 13:59:35]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-17 14:00:56]

Description was changed from

==========
Scale the deltaX/deltaY on the mouse wheel events to be consistent with the
compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441
==========

to

==========
Scale the deltaX/deltaY on the mouse wheel events to be consistent with the
compositor implementation.

As the page is zoomed in the scroll amount shouldn't increase. This issue
was discovered with the conversion to mouse wheel gestures which uses
gesture scrolls which are scaled; and the layout test was failing for those
cases.

BUG=595441

Committed: https://crrev.com/aa27c866cdef570e9788f9a93a3a3d87aa36ebce
Cr-Commit-Position: refs/heads/master@{#381702}
==========

[commit-bot: I haz the power, 2016-03-17 14:00:57]

Patchset 3 (id:??) landed as
https://crrev.com/aa27c866cdef570e9788f9a93a3a3d87aa36ebce
Cr-Commit-Position: refs/heads/master@{#381702}