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

Issue 595986 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Regression : Very small scrolling of content in drop down list is seen on mouse hover.

Reported by mni...@etouch.net, Mar 18 2016

Issue description

Chrome version : 51.0.2682.0 153a9028e5cb73e3c707b4884550499e2a08af5c-refs/heads/master@{#381839} (32/64 bit)
OS : Windows(Win 7 aero enabled)Linux.
Url : http://chess.mobialia.com/#run

Steps : 
1. Launch Chrome and open above url.
2. Now click on more options button on top RHS of page and go to settings.
3. Now click on 'Engine ELO' drop down list and hover the mouse on first content and observe

Actual : Flickering of content in drop down list is seen on mouse hover. 
Expected : Flickering of content in drop down list should not be seen on mouse hover.

This is a regression issue broken in 'M-49' and below is the manual regression and narrow bisect info:
Good build : 49.0.2574.0
Bad build : 49.0.2575.0

Narrow bisect :
https://chromium.googlesource.com/chromium/src/+log/d01d3720730602fdc8aeb19aa0b19ed6c5a5849c..1141951b9418bf41586643be378d7f5e5c87bfca?pretty=fuller&n=50

Suspecting : r361756 from Narrow bisect.

@tkent : Could you please help to reassign if your change is not the cause for this change.

Note : Issue is not seen on Mac OS.

 
Actual_video.mp4
605 KB Download
Expected_video.mp4
671 KB Download

Comment 1 by tkent@chromium.org, Mar 22 2016

Components: -Blink>Forms Blink>Forms>Select
Labels: -Pri-1 -M-51 Pri-3
Decrease the priority to P3.  It's a minor cosmetic issue.

Comment 2 by tkent@chromium.org, Mar 22 2016

Summary: Regression : Very small scrolling of content in drop down list is seen on mouse hover. (was: Regression : Flickering of content in drop down list is seen on mouse hover.)

Comment 3 by tkent@chromium.org, Jun 3 2016

Labels: PagePopupZoom
Owner: ----
Status: Available (was: Assigned)

Comment 4 by tkent@chromium.org, Dec 28 2016

Owner: tkent@chromium.org
Status: Started (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 4 2017

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

commit 421bed44b76d7743b573b4a6b313fa17813f8416
Author: tkent <tkent@chromium.org>
Date: Wed Jan 04 07:48:59 2017

SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px width.

[1] codereview.chromium.org/1454613002

BUG= 564506 , 570636 , 595986 

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

[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/fast/forms/select-popup/popup-menu-appearance-zoom090.html
[add] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/fast/forms/select-popup/popup-menu-appearance-zoom110-expected.png
[add] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/fast/forms/select-popup/popup-menu-appearance-zoom110-expected.txt
[add] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/fast/forms/select-popup/popup-menu-appearance-zoom110.html
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/platform/win/fast/forms/select-popup/popup-menu-appearance-zoom-expected.png
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/LayoutTests/platform/win/fast/forms/select-popup/popup-menu-appearance-zoom090-expected.png
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/Source/core/page/PagePopupClient.h
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/Source/web/PopupMenuImpl.cpp
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/Source/web/PopupMenuImpl.h
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/Source/web/resources/listPicker.js
[modify] https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416/third_party/WebKit/Source/web/resources/pickerCommon.js

Comment 6 by tkent@chromium.org, Jan 9 2017

Labels: M-57
Status: Fixed (was: Started)

Sign in to add a comment