New issue
Advanced search Search tips

Issue 920913 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Tool-tip appears behind the avatar menu on second instance.

Project Member Reported by aim...@virtusa.com, Jan 11

Issue description

Chrome 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!

 
Actual Result.mp4
468 KB View Download
Expected Result.mp4
354 KB View Download
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, 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

Status: Fixed (was: Started)

Sign in to add a comment