Regression: Vertical scroll down bar is not working on chrome://md-settings/searchEngines.
Reported by
aiman.an...@etouch.net,
Apr 18 2017
|
||||||||||
Issue descriptionChrome Version: 59.0.3071.9 (Official Build) (32-bit) (cohort: Stable)2ae1b9cb3ac16142bb504b4529323d14d0eb6f93-refs/branch-heads/3071@{#28}(32/64 Bit). OS: Windows(7,8,10), Linux(14.04 LTS), Mac(10.11.6, 10.12.1, 10.12). Steps to reproduce: 1.Launch Chrome, go to chrome://md-settings/searchEngines. 2. Resize the window, scroll down the page completely . 3. Try to scroll up (Using mouse wheel or clicking on up arrow) and observe. Actual Result: Scroll Bar is not working. Expected Result: Scroll Bar should work properly. This is regression issue broken in 'M 59' and will soon update the bisect info: Manual Bisect Info: Good Build 59.0.3067.0 Bad Build 59.0.3068.1
,
Apr 18 2017
Using the per-revision bisect providing the bisect results, Good build:59.0.3067.0 (Revision: 463157). Bad build: 59.0.3068.1 (Revision: 463474). You are probably looking for a change made after 463311 (known good), but no later than 463312 (first known bad). CHANGELOG URL: The script might not always return single CL as suspectas some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/8b7a7716bedcc5120cafc3df7ecb61aa542a1089..2aa395986fe5e88a8c6d867c5e6e663697c89854 From the CL above, assigning the issue to the concern owner @hcarmona - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Review-Url: https://codereview.chromium.org/2805213003 Thanks!
,
Apr 19 2017
Issue is still seen on Win 10 and Mac 10.12.4 using latest Canary 60.0.3074.0, and on Ubuntu 14.04 using 59.0.3071.9 as well. @hcarmona: Could you please look in to this issue and give us an update
,
Apr 19 2017
Investigating this. At first glance, this looks like it's happening b/c of my change in the 2nd iron list. It may not realize it needs to scroll....
,
Apr 20 2017
Issue 713591 has been merged into this issue.
,
Apr 27 2017
This seems to be working for me on Canary. Did anything get fixed? If we can't repro it anymore can it be closed?
,
Apr 27 2017
This only reproduces if you have no "Other search engines"
,
Apr 28 2017
Ya I can reproduce that. Removing the scroll listeners causes this to scroll so this isn't a Blink>Scroll issue. It purely is an app issue. Removing Blink>Scroll label.
,
May 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/df175df214a222e144783ec7cf776266fa8c974e commit df175df214a222e144783ec7cf776266fa8c974e Author: hcarmona <hcarmona@chromium.org> Date: Fri May 05 17:03:39 2017 MD-Settings - Update iron-list to fix scroll issue. Fixed in https://github.com/PolymerElements/iron-list/issues/438 BUG= 712544 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2858373003 Cr-Commit-Position: refs/heads/master@{#469685} [modify] https://crrev.com/df175df214a222e144783ec7cf776266fa8c974e/third_party/polymer/v1_0/bower.json [modify] https://crrev.com/df175df214a222e144783ec7cf776266fa8c974e/third_party/polymer/v1_0/components-chromium/iron-list/bower.json [modify] https://crrev.com/df175df214a222e144783ec7cf776266fa8c974e/third_party/polymer/v1_0/components-chromium/iron-list/iron-list-extracted.js [modify] https://crrev.com/df175df214a222e144783ec7cf776266fa8c974e/third_party/polymer/v1_0/components_summary.txt
,
May 8 2017
Requesting merge for http://crrev.com/2858373003 Change is small (updates iron-list) and fixes a bad scrolling issue Change has been in canary for a few days and is working well
,
May 8 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1867c8fdf276d3045d79f720a3057350864b6dbe commit 1867c8fdf276d3045d79f720a3057350864b6dbe Author: Hector Carmona <hcarmona@chromium.org> Date: Mon May 08 18:44:17 2017 MD-Settings - Update iron-list to fix scroll issue. Fixed in https://github.com/PolymerElements/iron-list/issues/438 BUG= 712544 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2858373003 Cr-Commit-Position: refs/heads/master@{#469685} (cherry picked from commit df175df214a222e144783ec7cf776266fa8c974e) Review-Url: https://codereview.chromium.org/2867053003 . Cr-Commit-Position: refs/branch-heads/3071@{#453} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} [modify] https://crrev.com/1867c8fdf276d3045d79f720a3057350864b6dbe/third_party/polymer/v1_0/bower.json [modify] https://crrev.com/1867c8fdf276d3045d79f720a3057350864b6dbe/third_party/polymer/v1_0/components-chromium/iron-list/bower.json [modify] https://crrev.com/1867c8fdf276d3045d79f720a3057350864b6dbe/third_party/polymer/v1_0/components-chromium/iron-list/iron-list-extracted.js [modify] https://crrev.com/1867c8fdf276d3045d79f720a3057350864b6dbe/third_party/polymer/v1_0/components_summary.txt
,
May 9 2017
This was merged, marking as fixed.
,
May 10 2017
Verified this issue on Windows-10, Mac OS 10.12.4 and Ubuntu 14.04 using chrome latest beta M59-59.0.3071.47 by following steps mentioned in the original comment. Observed the page chrome://md-settings/searchEngines able to scroll up using mouse wheel and clicking on up arrow button as expected, Hence adding TE-Verified label. |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by aiman.an...@etouch.net
, Apr 18 20171.2 MB
1.2 MB View Download
1.2 MB
1.2 MB View Download