Issue metadata
Sign in to add a comment
|
Regression: Tool-tip appears behind the avatar menu on second instance. |
||||||||||||||||||||||
Issue descriptionChrome Version: 73.0.3667.0 (Official Build) e908663b57104ae7163ae941e16549e2a8ad65f5-refs/branch-heads/3667@{#1}(32/64-bit). OS: Win(7,8,8.1,10), What steps will reproduce the problem? 1. Launch chrome and Open NTP. 2. Click on Avatar Icon in omnibox to open Avatar overlay and now just mouse hover on wrench icon to see tool-tip. 3. Hit 'ESC' and again repeat Step-2. 4. Observe. Actual Result: Tool-tip appears behind the avatar menu on second instance. Expected Result: Tool-tip should not be behind the avatar menu on mouse-hovering. This is a regression issue broken in M-71 and below is the per-revision-bisect-info: Good Build : 71.0.3569.0(Revision:596016) Bad Build : 71.0.3570.0(Revision:596428) You are probably looking for a change made after 596274 (known good), but no later than 596275 (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/2b798e975c6cdb02dee93bdaf184276ad21d4be9..8abd68396d3599abe18fa5797ec6dff2f8f68fad Suspect: https://chromium.googlesource.com/chromium/src/+/8abd68396d3599abe18fa5797ec6dff2f8f68fad dfried: 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 not seen on Mac(10.13.1, 10.13.6, 10.14.2) and Linux(14.04 LTS) OS. 2. Issue is also seen on Stable #71.0.3578.98 Beta #72.0.3626.53 and Dev #73.0.3664.3 build. Kindly refer the screen cast for reference Thank You!
,
Jan 14
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/65deacdec80d85b0114e3b905b7ed303bba71d8a commit 65deacdec80d85b0114e3b905b7ed303bba71d8a Author: Dana Fried <dfried@chromium.org> Date: Mon Jan 14 20:16:15 2019 Bring tooltip to front on Show() This code was erroneously removed as spurious in a previous patch to fix font sizing on DPI-scaled displays. It's back now. Bug: 920913 Change-Id: Ica0cd4e98ca2227566becc41a2620e771bd06775 Reviewed-on: https://chromium-review.googlesource.com/c/1408042 Reviewed-by: Scott Violet <sky@chromium.org> Commit-Queue: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#622570} [modify] https://crrev.com/65deacdec80d85b0114e3b905b7ed303bba71d8a/ui/views/corewm/tooltip_win.cc
,
Jan 14
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dfried@chromium.org
, Jan 12