New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 762386 link

Starred by 16 users

Touch Scroll is not smooth on Stable build# 61.0.3163.79

Project Member Reported by ranjitkan@chromium.org, Sep 6 2017

Issue description

Chrome Version: 61.0.3163.79
OS: Windows 10 Touch device

What steps will reproduce the problem?
(1) Launch chrome version# 61.0.3163.79
(2) Navigate to a Wiki page or a page which has a some large content to scroll
(3) Try scrolling using touch

What is the expected result?
Touch scroll should be smooth

What happens instead?
Scroll is not smooth using touch.

Working on Bisect and will update the details soon.

 
Cc: sky@chromium.org dtapu...@chromium.org
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: lanwei@chromium.org
Status: Assigned (was: Untriaged)
Below are the bisect details for the same:

Bisect Info:
============
Good build:61.0.3143.0 
Bad build: 61.0.3144.0 

You are probably looking for a change made after 482976 (known good), but no later than 482977 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/a4f88bdad6d5b47e1f9f022ecda09b0909af6852..4c69348ec2a2721e25ae02b9bc75aff88e38973c

@lanwei: Assigning to you, kindly take a look into it. Please help us to find an owner if not with respect to your change. Also tagging with blocker label, please undo if not the case.
Owner: dtapu...@chromium.org
Is there a trace?
Please test on canary I believe this is fixed by crrev.com/498498.

I'll try to prepare a build locally with it and test it.
Related Comments from users on chrome release blog post - https://chromereleases.googleblog.com/2017/09/stable-channel-update-for-desktop.html#gpluscomments

* We lost scroll inertia for touch-enabled Windows computers.
* touch scroll does not work
* after this update, body.scrollTop and body.scrollLeft is gone.
Fix: use document.documentElement
This will ultimately be a duplicate of  issue 747642 
Owner: girard@chromium.org
Cc: jainabhi...@chromium.org
Labels: Hotlist-ConOps Hotlist-ConOps-Channel-Beta Hotlist-ConOps-Source-Feedback
Some user feedback
https://listnr/report/72493033301 - Windows NT: 10.0.15063
Chrome is not scrolling smoothly in Windows 10.  Usually a touchscreen "flick" gets the vertical scroll to coast smoothly.  I checked The Edge browser and File Explorer and both are scrolling easily and smoothly with the same gesture.

https://listnr/report/72508420255 - Windows NT: 10.0.15063
Just updated chrome browser and the touch scrolling has been messed up.  does not continue to scroll based on velocity of touch.

https://listnr/report/72509072568 - Windows NT: 6.1.7601 SP1
A couple of webpages that scroll fine in other browsers and used to scroll fine in Chrome (via the scroll bar), are no longer scrollable in the latest Chrome release.  The issue could fix itself if I reboot this computer but that is such a hassle when you are in the middle of several things and have many things open...
Can this be marked as dupe of  bug 747642  which is currently marked as fixed and M61 merge is requested - https://bugs.chromium.org/p/chromium/issues/detail?id=747642#c10?
I have the same issue since Release-61. Is there any way to revert to Release-60? This behaviour has destroyed my workflow. The current beta build does not fix the issue.
Labels: Merge-Request-61
As discussed in email thread, I am requesting to merge a patch that disables pointer events for touch (they remain enabled for stylus).
Project Member

Comment 12 by sheriffbot@chromium.org, Sep 7 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: Request affecting a post-stable build
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Merge-Request-62
Patch is at  https://chromium-review.googlesource.com/655645
Project Member

Comment 14 by bugdroid1@chromium.org, Sep 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e976a3897d7a8abbbacce4c2622cc5ecf5a7b067

commit e976a3897d7a8abbbacce4c2622cc5ecf5a7b067
Author: Eugene Girard <girard@chromium.org>
Date: Fri Sep 08 04:26:55 2017

Reverting pointer events for touch

Code originally landed in https://codereview.chromium.org/2904113002
Several regressions were noted:  crbug.com/762665 ,  crbug.com/762386 , and  crbug.com/762910 

Bug:  762386 
Change-Id: If496431076c5a37e324dd70e3794ad72aeb9e913
Reviewed-on: https://chromium-review.googlesource.com/655645
Commit-Queue: Eugene Girard <girard@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Eugene Girard <girard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500507}
[modify] https://crrev.com/e976a3897d7a8abbbacce4c2622cc5ecf5a7b067/ui/views/win/hwnd_message_handler.cc

Labels: -Merge-Request-62 Merge-Approved-62
Project Member

Comment 16 by bugdroid1@chromium.org, Sep 8 2017

Labels: merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9859bd4ce07c10ea78e419294fbe242c39d2d270

commit 9859bd4ce07c10ea78e419294fbe242c39d2d270
Author: Eugene Girard <girard@chromium.org>
Date: Fri Sep 08 20:37:57 2017

Reverting pointer events for touch

Code originally landed in https://codereview.chromium.org/2904113002
Several regressions were noted:  crbug.com/762665 ,  crbug.com/762386 , and  crbug.com/762910 

TBR=girard@chromium.org

(cherry picked from commit e976a3897d7a8abbbacce4c2622cc5ecf5a7b067)

Bug:  762386 
Change-Id: If496431076c5a37e324dd70e3794ad72aeb9e913
Reviewed-on: https://chromium-review.googlesource.com/655645
Commit-Queue: Eugene Girard <girard@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Eugene Girard <girard@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#500507}
Reviewed-on: https://chromium-review.googlesource.com/658220
Cr-Commit-Position: refs/branch-heads/3163@{#1147}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/9859bd4ce07c10ea78e419294fbe242c39d2d270/ui/views/win/hwnd_message_handler.cc

Project Member

Comment 17 by bugdroid1@chromium.org, Sep 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2954414d71ce30cc5e890c7e20272ed0e4d2c015

commit 2954414d71ce30cc5e890c7e20272ed0e4d2c015
Author: Eugene Girard <girard@chromium.org>
Date: Fri Sep 08 21:11:24 2017

Revert "Reverting pointer events for touch"

This reverts commit 9859bd4ce07c10ea78e419294fbe242c39d2d270.

R=sadrul@chromium.org

Bug:  762386 
Change-Id: I8fcc9721cb255379e38fcbc09be26f6281ab5007
Reviewed-on: https://chromium-review.googlesource.com/658223
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#1148}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/2954414d71ce30cc5e890c7e20272ed0e4d2c015/ui/views/win/hwnd_message_handler.cc

NextAction: 2017-09-11
Thank you girard@ for quick revert from M61. Please wait until M61 approval on Monday by then change will be tested/baked in Canary.
Project Member

Comment 19 by bugdroid1@chromium.org, Sep 8 2017

Labels: -merge-approved-62 merge-merged-3202
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e3f924cc2397b8847ae456f659e7794f979b2a5a

commit e3f924cc2397b8847ae456f659e7794f979b2a5a
Author: Eugene Girard <girard@chromium.org>
Date: Fri Sep 08 21:47:29 2017

Reverting pointer events for touch

Code originally landed in https://codereview.chromium.org/2904113002
Several regressions were noted:  crbug.com/762665 ,  crbug.com/762386 , and  crbug.com/762910 

TBR=girard@chromium.org

(cherry picked from commit e976a3897d7a8abbbacce4c2622cc5ecf5a7b067)

Bug:  762386 
Change-Id: Idebfca6fa21bfd991b45605e2380d0d8e832787f
Reviewed-on: https://chromium-review.googlesource.com/655645
Commit-Queue: Eugene Girard <girard@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Eugene Girard <girard@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#500507}
Reviewed-on: https://chromium-review.googlesource.com/658285
Cr-Commit-Position: refs/branch-heads/3202@{#98}
Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098}
[modify] https://crrev.com/e3f924cc2397b8847ae456f659e7794f979b2a5a/ui/views/win/hwnd_message_handler.cc

Labels: TE-Verified-M63 TE-Verified-63.0.3212.0
Rechecked this issue on Windows 10 touch device using chrome version 63.0.3212.0 and fix is working as intended. Able to scroll the pages using touch smoothly. Adding TE-verified labels for M63
The NextAction date has arrived: 2017-09-11
Labels: -Merge-Review-61 Merge-Approved-61
Prudhvi tested and confirmed testing on Win7 touchscreen looks good.  Approving merge to M61 branch 3163. Please merge ASAP. Thank you.
Project Member

Comment 23 by bugdroid1@chromium.org, Sep 11 2017

Labels: -merge-approved-61
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/42004c5c4835b70c753541df8474b8aa4e6064f2

commit 42004c5c4835b70c753541df8474b8aa4e6064f2
Author: Eugene Girard <girard@chromium.org>
Date: Mon Sep 11 20:58:07 2017

Reverting pointer events for touch

Code originally landed in https://codereview.chromium.org/2904113002
Several regressions were noted:  crbug.com/762665 ,  crbug.com/762386 , and  crbug.com/762910 

(cherry picked from commit e976a3897d7a8abbbacce4c2622cc5ecf5a7b067)

TBR: sky
Bug:  762386 
Change-Id: I416b2cea7b36265f7626505d2d236bf3885751a4
Reviewed-on: https://chromium-review.googlesource.com/655645
Commit-Queue: Eugene Girard <girard@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Eugene Girard <girard@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#500507}
Reviewed-on: https://chromium-review.googlesource.com/660990
Cr-Commit-Position: refs/branch-heads/3163@{#1163}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/42004c5c4835b70c753541df8474b8aa4e6064f2/ui/views/win/hwnd_message_handler.cc

Labels: TE-Verified-M62 TE-Verified-62.0.3202.18
Rechecked this issue on Windows 10 touch device with chrome dev version 62.0.3202.18, fix is working as intended. Able to scroll the webpages via touch smoothly. Adding TE-verified labels for M62
Status: Fixed (was: Assigned)
Labels: TE-Verified-61.0.3163.91 TE-Verified-M61
Rechecked this issue on Windows 10 touch device using chrome version 61.0.3163.91 and fix is working as intended. Able to scroll the pages using touch smoothly. Adding TE-verified labels for M61

Comment 27 by ma10...@gmail.com, Sep 14 2017

THANK YOU !!!!

Comment 28 by ma10...@gmail.com, Sep 14 2017

Dear Sir:

             It worked for a few seconds. Then it stopped
working.
This patch is now pushing out to stable channel in version 61.0.3163.91 for Desktop (Win,Mac & Linux).

Comment 30 by ma10...@gmail.com, Sep 14 2017

Thank you so very much! It fixed the smooth scrolling
problem.
Thank you for fixing the bug. Requesting postmortem for this please see go/chrome-postmortems for the process to follow.

Sign in to add a comment