Issue metadata
Sign in to add a comment
|
Touch Scroll is not smooth on Stable build# 61.0.3163.79 |
||||||||||||||||||||||
Issue descriptionChrome 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.
,
Sep 6 2017
Is there a trace?
,
Sep 6 2017
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.
,
Sep 6 2017
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
,
Sep 6 2017
This will ultimately be a duplicate of issue 747642
,
Sep 6 2017
,
Sep 6 2017
,
Sep 6 2017
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...
,
Sep 7 2017
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?
,
Sep 7 2017
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.
,
Sep 7 2017
As discussed in email thread, I am requesting to merge a patch that disables pointer events for touch (they remain enabled for stylus).
,
Sep 7 2017
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
,
Sep 7 2017
,
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
,
Sep 8 2017
,
Sep 8 2017
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
,
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
,
Sep 8 2017
Thank you girard@ for quick revert from M61. Please wait until M61 approval on Monday by then change will be tested/baked in Canary.
,
Sep 8 2017
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
,
Sep 11 2017
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
,
Sep 11 2017
The NextAction date has arrived: 2017-09-11
,
Sep 11 2017
Prudhvi tested and confirmed testing on Win7 touchscreen looks good. Approving merge to M61 branch 3163. Please merge ASAP. Thank you.
,
Sep 11 2017
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
,
Sep 12 2017
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
,
Sep 13 2017
,
Sep 14 2017
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
,
Sep 14 2017
THANK YOU !!!!
,
Sep 14 2017
Dear Sir:
It worked for a few seconds. Then it stopped
working.
,
Sep 14 2017
This patch is now pushing out to stable channel in version 61.0.3163.91 for Desktop (Win,Mac & Linux).
,
Sep 14 2017
Thank you so very much! It fixed the smooth scrolling problem.
,
Sep 14 2017
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 |
|||||||||||||||||||||||
Comment 1 by ranjitkan@chromium.org
, Sep 6 2017Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: lanwei@chromium.org
Status: Assigned (was: Untriaged)