Incorrect tooltip is seen for New Tab page on selecting Mute tab option.
Reported by
vvishwak...@etouch.net,
Mar 31 2016
|
||||
Issue descriptionVersion: 51.0.2695.0 (Official Build) 723dfe66b1985ff2178590b15d1cc63911300737-refs/heads/master@{#384157} (64-bit) OS: Mac What steps will reproduce the problem? 1) Launch chrome, open a New tab, right click on 'New Tab' button and select 'Mute tab' option. 2) Mouse hover on tab immediately after selecting 'Mute tab' option. 3) Observe the tooltip. 'New Tab' tooltip is seen on mouse hover. 'New Tab. This tab's audio is being muted' tooltip should be seen on mouse hover. This is a Non-Regression issue seen from M-46 (Chrome version: 46.0.2480.0) Note: 1) Issue is not seen on Windows and Linux OS. 2)
,
Mar 31 2016
Untriaging it so that it gets addressed.
,
Apr 1 2016
[mac triage] this seems to fix it https://codereview.chromium.org/1855503002/
,
Apr 18 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cad10375b43638e45bb4e7630e71313b59a13842 commit cad10375b43638e45bb4e7630e71313b59a13842 Author: tapted <tapted@chromium.org> Date: Mon Apr 18 01:14:47 2016 Mac: Update the tooltip view tooltip when a tab's title changes. Currently, when muting a tab, the tooltip only updates when hovering off of a tab, then hovering back on. This is because only the tab view's tooltip updates, but there's an optimization on mac that involves adding a separate transparent view to handle tab tooltips. If that tooltip view is for the changed tab, ensure its tooltip is updated as well. BUG= 599401 TEST=On Mac, reveal a tab's tooltip, then mute/unmute a tab and immediately hover over it again. Ensure the tooltip is updated appropriately. Review URL: https://codereview.chromium.org/1855503002 Cr-Commit-Position: refs/heads/master@{#387840} [modify] https://crrev.com/cad10375b43638e45bb4e7630e71313b59a13842/chrome/browser/ui/cocoa/tabs/tab_strip_controller.mm [modify] https://crrev.com/cad10375b43638e45bb4e7630e71313b59a13842/chrome/browser/ui/cocoa/tabs/tab_strip_controller_unittest.mm
,
Apr 18 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by vvishwak...@etouch.net
, Mar 31 2016