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

Issue 650557 link

Starred by 1 user
Status: Fixed
Owner:
█ dschuyler@chromium.org
Last visit > 30 days ago
Closed: Sep 2016
Cc:
nyerramilli@chromium.org
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : 'I-beam pointer' is seen instead of 'hand pointer' on mouse hover on site URLs in siteSettings/all.

Reported by yfulgaon...@etouch.net, Sep 27 2016 
Chrome Version : 55.0.2873.0 (Official Build) 4e1f4cd0a29061dc75bc76aef5212847c2ca132a-refs/heads/master@{#421052} (32/64-bit)
OS: Mac(10.10.5)(10.11.5), Windows(7,8,8.1,10), Linux(14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome, navigate to chrome://md-settings/siteSettings and click on ‘All sites’.
2. Now hover mouse pointer on any of the site URLs and observe the mouse pointer.

Actual : I-beam pointer is seen on mouse hover on site URLs in siteSettings/all.
Expected : Instead, hand pointer should be seen on mouse hover on site URLs in siteSettings/all.

This is a regression issue broken in ‘M-55’, below is the Manual Regression and Narrow Bisect info. 
Good build: 55.0.2868.0
Bad build: 55.0.2869.0
Actual_site_settings.mov
3.9 MB Download
Expected_site_settings.mov
2.8 MB Download

Comment 1 by msrchandra@chromium.org, Sep 27 2016

Labels: -hasbisect hasbisect-per-revision ReleaseBlock-Beta
Owner: finnur@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 55.0.2868.0 (Revision: 420217).
Bad build: 55.0.2869.0 (Revision: 420536).

You are probably looking for a change made after 420301 (known good), but no later than 420302 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/3b3e90443440cecbc3d24a045819b3d14e0b17c6..b0fca5c853d275637871e0ac86f84953f2071346

Note: Adding RB Label as this is a recent Regression. Please remove if not required.
@finnur -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
Thank You.

Comment 2 by finnur@chromium.org, Sep 27 2016

Owner: dschuyler@chromium.org
Dave already has a fixed lined up for this.
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 27 2016 

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

commit a241e7aac6b340112403a9a9f4d2919a41972839
Author: dschuyler <dschuyler@chromium.org>
Date: Tue Sep 27 13:58:05 2016

[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG= 650557 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2359403002
Cr-Commit-Position: refs/heads/master@{#421197}

[modify] https://crrev.com/a241e7aac6b340112403a9a9f4d2919a41972839/chrome/browser/resources/settings/site_settings/site_list.html

Comment 4 by dschuyler@chromium.org, Sep 29 2016

Status: Fixed (was: Assigned)

Comment 5 by kavvaru@chromium.org, Oct 4 2016

Labels: TE-Verified-M55 TE-Verified-55.0.2880.0
Tested the issue on windows 7, Mac 10.11.6 using chrome version 55.0.2880.0.Seeing hand pointer on URLs in siteSettings. Please find the attached screen cast for the same.

Adding TE-Verified labels.

Note:: Issue verified on 55.0.2879.0 for Linux as the 55.0.2880.0 build got failed for linux due to issue 652581.

Thanks,
650557.mp4
841 KB View Download
►

Sign in to add a comment