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

Issue 820923 link

Starred by 2 users
Status: Duplicate
Merged: issue 776205
Owner:
█ yoshiki@chromium.org
Last visit > 30 days ago
Closed: Mar 2018
Cc:
ajha@chromium.org
tetsui@chromium.org
█ brajkumar@chromium.org
█ megumihattori@google.com
steve...@chromium.org
peter@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression

Blocked on:
issue 776205


Sign in to add a comment

Regression : Arrow pointer is seen instead of Hand pointer on hovering mouse pointer at bad password notification

Project Member Reported by mmanchala@chromium.org, Mar 12 2018 
Chrome Version:  66.0.3359.28/10452.8.0 dev-channel Daisy,Reks and Kip
OS: Chrome

What steps will reproduce the problem?
(1)Recover build>> In connect to Network overlay, give any wrong password nearer to the correct password and click on 'CONNECT'
(2)After few seconds observe bad password notification is seen -> Now hover mouse pointer on notification and observe Arrow pointer is seen instead of Hand pointer(as it is clickable)
(Please refer Video)

Expected:  Hand pointer should be seen on hovering mouse pointer on bad password notification as it is clickable
Actual: Instead  Arrow pointer is seen 

This is a Regression issue as same is working fine on M-63 

@yoshiki: Please confirm the issue

Note :
1.Issue is working fine on M-62
2.Issue is also seen on M-64 and M-65
Actual_ArrowPointer.mp4
9.9 MB View Download
Actual_Arrowpointer.jpg
766 KB View Download
Expected_HandPointer.mp4
8.5 MB View Download
Expected_HandPointer.jpg
501 KB View Download

Comment 1 by yoshiki@chromium.org, Mar 13 2018

Status: Started (was: Assigned)
As I investigated, it's a regression by https://chromium.googlesource.com/chromium/src/+/e43ca886e2a7c023f6237c5390a98c7cbc886ed5%5E%21/

HandleNotificationClickDelegate::HasClickListener returned true if callback existed, so that we had to add "set_clickable(true)" to all the code with CreateSystemNotification and HandleNotificationClickDelegate. But some of code lacks it.

Let me fix.

Comment 3 by yoshiki@chromium.org, Mar 15 2018

Blockedon: 776205

Comment 4 by yoshiki@chromium.org, Mar 22 2018

Mergedinto: 776205
Status: Duplicate (was: Started)
►

Sign in to add a comment