Remove JoystickScrollProvider

Remove JoystickScrollProvider

JoystickScrollProvider introduced a new scrolling mode that
required ContentViewCore to keep state (last motion event position
in physical pixels). According to the bug 650351 the right way
to fix it is to rewrite the joystick scroll.

Thus, removing the current implementation.

BUG=650351, 620929

[Tima Vaisburd, 2016-09-27 21:07:56]

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

[commit-bot: I haz the power, 2016-09-27 21:08:26]

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-09-27 21:09:35]

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

[commit-bot: I haz the power, 2016-09-27 21:09:36]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Tima Vaisburd, 2016-09-27 21:34:54]

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

[commit-bot: I haz the power, 2016-09-27 21:35:11]

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-09-27 22:30:17]

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

[commit-bot: I haz the power, 2016-09-27 22:30:18]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Tima Vaisburd, 2016-09-27 23:43:44]

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

[commit-bot: I haz the power, 2016-09-27 23:44:14]

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

[Tima Vaisburd, 2016-09-27 23:47:20]

timav@chromium.org changed reviewers:
+ boliu@chromium.org, mustaq@chromium.org, rbyers@chromium.org,
tedchoc@chromium.org

[Tima Vaisburd, 2016-09-27 23:47:20]

PTAL: I went ahead and removed current joystick scroll completely. Please advise
whether it is desirable or too drastic.

[commit-bot: I haz the power, 2016-09-28 01:47:26]

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

[commit-bot: I haz the power, 2016-09-28 01:47:27]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Tima Vaisburd, 2016-09-28 22:56:27]

On 2016/09/27 23:52:03, Tima Vaisburd wrote:

Closing per https://bugs.chromium.org/p/chromium/issues/detail?id=650351#c4 :
Deletion is premature, we agreed with NVidia on short-term maintenance.

aelias@: last motion event offset could be eliminated by generating synthetic
mousewheel events instead of touch events.

[Tima Vaisburd, 2016-09-28 22:56:44]

Description was changed from

==========
Remove JoystickScrollProvider

JoystickScrollProvider introduced a new scrolling mode that
required ContentViewCore to keep state (last motion event position
in physical pixels). According to the bug 650351 the right way
to fix it is to rewrite the joystick scroll.

Thus, removing the current implementation.

BUG=650351, 620929
==========

to

==========
Remove JoystickScrollProvider

JoystickScrollProvider introduced a new scrolling mode that
required ContentViewCore to keep state (last motion event position
in physical pixels). According to the bug 650351 the right way
to fix it is to rewrite the joystick scroll.

Thus, removing the current implementation.

BUG=650351, 620929
==========