New issue
Advanced search Search tips

Issue 909580 link

Starred by 2 users
Status: Fixed
Owner:
lanwei@chromium.org
Closed: Jan 7
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Blue selection highlight doesn't appear on scrolling over the options in dropdown list

Reported by khushal....@etouch.net, Nov 28 
Chrome Version: 71.0.3578.75 (Official Build) Revision 06ef00b5279f93f8e0c1e73acedd49d7dcc09767-refs/branch-heads/3578@{#836} (32/64-bit)

OS: Windows (7, 8, 8.1, 10) & Linux (14.04 LTS)

What steps will reproduce the problem?
(1) Launch chrome, navigate to chrome://settings/autofill/ page and click on 'Add' button ('Add address' overlay will open).
(2) Scroll down and click on 'Country/Region' dropdown.
(3) Place the mouse cursor over any option from the dropdown list (Blue highlight selection will appear over the option).
(4) Now without moving the mouse scroll down the dropdown list.
(5) Observe.

Actual Result: Blue selection highlight doesn't appear on scrolling over the options in dropdown list.

Expected Result: Blue selection highlight should appear wherever the mouse cursor is placed when the options are scrolled.

This is a Regression issue seen from 'M-61' and providing the bisect info below:
Good Build: 61.0.3139.0 (Revision: 481758)
Bad Build:  61.0.3140.0 (Revision: 482096)

You are probably looking for a change made after 481918 (known good), but no later than 481919 (first known bad).

CHANGE-LOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  
https://chromium.googlesource.com/chromium/src/+log/18c51c84da2f5321c0a3252c2776835b79d83827..5c76d364d68a5c43fe2f5c37e32aba3148b9a300

Suspect: https://chromium.googlesource.com/chromium/src/+/5c76d364d68a5c43fe2f5c37e32aba3148b9a300

@lanwei: 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.

NOTE:
1. Issue is also seen on M-70 Stable (build #70.0.3538.110), M-71 Beta (build #71.0.3578.62) and M-72 Dev (build #72.0.3610.2).
2. Issue is not seen on Mac (10.13.1, 10.13.6, 10.14.2).

Kindly refer the attached screen-cast.

Thank You..!!
Actual Video.mp4
827 KB View Download
Expected Video.mp4
894 KB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 10 

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

commit 6be942365613e726a07925f91cf0811ef4a3a576
Author: Lan Wei <lanwei@chromium.org>
Date: Mon Dec 10 15:38:26 2018

Update hover when scrolling over the options in dropdown list

When we scroll over the options in dropdown list, we cannot update the
hover effect by highlighting the option under mouse cursor blue. The
reason is when we scroll, the mouse cursor does not really move, but we
want to update the hover and send right mouse boundary events, we send
a fake mouse move event, but we do not send to DOM. The option list has
an default mouse move listener to update the hover in listPicker.js,
but we do not send mouse move for this case.

If we want to fix it, we should use a mouseover listener to update
the hover instead. Then the blue selection highlight will appear
wherever the mouse cursor is placed when the options are scrolled, but
the mouse cursor does not move.

Bug:  909580 
Change-Id: Iec0603016a4f674c4420eac5eb01b36aaefccbf0
Reviewed-on: https://chromium-review.googlesource.com/c/1362241
Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Commit-Queue: Lan Wei <lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#615127}
[modify] https://crrev.com/6be942365613e726a07925f91cf0811ef4a3a576/third_party/blink/renderer/core/html/forms/resources/listPicker.js

Comment 2 by khush...@virtusa.com, Dec 11

Labels: TE-Verified-M73 TE-Verified-73.0.3637.0
Update:

Rechecked the above issue on Windows (7, 8, 8.1, 10) & Linux (14.04 LTS) using latest Canary version #73.0.3637.0 and the issue is FIXED.
Hence, adding TE verified labels.

Please refer the attached screen-cast.

Thank You..!!
Fixed Video.mp4
864 KB View Download

Comment 3 by lanwei@chromium.org, Jan 7

Status: Fixed (was: Assigned)
►

Sign in to add a comment