Regression: [Md Settings] unnecessary I-beam pointer appears instead of hand pointer under 'Privacy and Security' section.
Reported by
dchau...@etouch.net,
Jan 12 2017
|
|||||||||
Issue descriptionChrome Version: 57.0.2979.0 (Official Build)1916748e2f13a9f80080d010a9adc6ba74028a59-refs/heads/master@{#443120} 32/64-bit. OS: Windows(7,8,10), Mac (10.11.6, 10.12.1), Linux (14.04 LTS) What steps will reproduce the problem? 1. Launch chrome, navigate to chrome://md-settings page and go to 'Advanced' section. 2. Hover the mouse pointer on 'Automatically report details of possible security incident to Google' and'Automatically send usage statistics and crash reports to Google' option under 'Privacy and Security' section and observe. On hovering the mouse pointer, unnecessary I-beam pointer appears instead of hand pointer. On hovering the mouse pointer, Hand pointer should be seen instead of I-beam pointer. This is a regression issue, broken in M-57 series, will soon update other info.
,
Jan 12 2017
Bisect Info: =========== Good build : 57.0.2978.0, Revision Range -442756 Bad build : 57.0.2979.0, Revision Range -443120 After executing the per-revision bisect script , i got the following CL's between good and bad build versions =========================================== https://chromium.googlesource.com/chromium/src/+log/a8f55a43d7da20364992515885fc25c39ca97e8d..0811dbe5577e389b35e8bc42ff9d19da756de89c The suspecting Change Log is : ----------- https://chromium.googlesource.com/chromium/src/+/0811dbe5577e389b35e8bc42ff9d19da756de89c From the above CL suspecting the below change --------------------------- Review-Url: https://codereview.chromium.org/2617533003 dbeam@- Could you please look into this issue, if it's related to your change? if not could you please help us to reassign this issue to the right owner.
,
Jan 20 2017
Friendly ping!! dbeam@, Still we are able to reproduce the issue on windows 7 with latest Canary#57.0.2986.0. Could you please take a look & update the thread. Thank you.
,
Feb 8 2017
,
Feb 8 2017
,
Feb 8 2017
,
Feb 8 2017
,
Feb 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3245561158dde6808bd24b3322304369af52abac commit 3245561158dde6808bd24b3322304369af52abac Author: dbeam <dbeam@chromium.org> Date: Tue Feb 14 08:49:01 2017 MD Settings: change how tap is handled on custom toggle rows R=scottchen@chromium.org BUG= 680406 ,689763 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2684853003 Cr-Commit-Position: refs/heads/master@{#450293} [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/controls/settings_boolean_control_behavior.js [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/controls/settings_checkbox.html [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/controls/settings_toggle_button.html [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/controls/settings_toggle_button.js [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/privacy_page/compiled_resources2.gyp [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/privacy_page/privacy_page.html [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/browser/resources/settings/privacy_page/privacy_page.js [modify] https://crrev.com/3245561158dde6808bd24b3322304369af52abac/chrome/test/data/webui/settings/metrics_reporting_tests.js
,
Feb 14 2017
,
Feb 16 2017
Verified this issue on Window 10, Ubuntu 14.04 and mac 10.12.3 with chrome #58.0.3013.3 Observed that Hand pointer is only shown for 'Automatically report details of possible security incident to Google' not for 'Automatically send usage statistics and crash reports to Google' Could you please look into it and let us know your observations
,
Feb 16 2017
that's because you can't change "send usage stats" because it's managed by your administrator (so we don't show a hand)
,
Feb 17 2017
As Per comment #12, adding TE verified labels. Thank You... |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by dchau...@etouch.net
, Jan 12 20171.9 MB
1.9 MB View Download
1.1 MB
1.1 MB View Download