Issue metadata
Sign in to add a comment
|
Regression: Header of chrome://downloads page appears chopped on toggling to full screen mode.
Reported by
lpa...@etouch.net,
Jun 9 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 53.0.2763.0 (Official Build) dd933bda0baa6a13ab0120f0056a2b783e459efb-refs/heads/master@{#398752} (32/64-bit) OS: Mac (10.10.5 , 10.11.4),Mac Retina (10.11.4) Steps: 1. Launch chrome and navigate to chrome://downloads. 2. Enter full screen then restore and again enter full screen. 3. Observe. Actual Behaviour: The header of downloads bar is seen chopped after toggling to fullscreen. Expected Behaviour: The header of the downloads bar should be properly visible. This is a regression issue broken in ‘M-52’ and below is the manual regression range. Good Build: 52.0.2734.0 Bad Build: 52.0.2736.0 Narrow Bisect: https://chromium.googlesource.com/chromium/src/+log/552dbdc4c66ae49753f84c892d5e3ff54daeb1ab..c5e032910d4c8119f8c40df507fda64e5fec11a0?pretty=fuller&n=100 Suspecting: r393642 Note: 1.Issue not seen on Win & Linux OS.
,
Jun 9 2016
,
Jun 14 2016
Unable to reproduce the issue on mac 10.11 chrome version 53.0.2767.0 - The header of downloads bar is properly displayed lpanse@etouch.net, Could you please recheck and confirm
,
Jun 14 2016
With response to comment #3, Issue is reproducible on latest canary version 53.0.2767.0 for OS version: Mac 10.10.5
,
Jun 16 2016
A friendly reminder that M52 Stable is launching soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch by July 12. All changes MUST be merged into the release branch by 5pm on July 15 to make into the desktop Stable final build cut. Thank you!
,
Jun 17 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/04542485cce24e64e794c1dfd47d97fa5fbdf20c commit 04542485cce24e64e794c1dfd47d97fa5fbdf20c Author: spqchan <spqchan@chromium.org> Date: Fri Jun 17 17:33:11 2016 [Mac] Fix Toolbar glimpse during fullscreen transition BUG= 618594 , 615810 Review-Url: https://codereview.chromium.org/2078733002 Cr-Commit-Position: refs/heads/master@{#400450} [modify] https://crrev.com/04542485cce24e64e794c1dfd47d97fa5fbdf20c/chrome/browser/ui/cocoa/browser_window_controller.h [modify] https://crrev.com/04542485cce24e64e794c1dfd47d97fa5fbdf20c/chrome/browser/ui/cocoa/browser_window_controller.mm [modify] https://crrev.com/04542485cce24e64e794c1dfd47d97fa5fbdf20c/chrome/browser/ui/cocoa/presentation_mode_controller.h [modify] https://crrev.com/04542485cce24e64e794c1dfd47d97fa5fbdf20c/chrome/browser/ui/cocoa/presentation_mode_controller.mm
,
Jun 17 2016
,
Jun 18 2016
Your change meets the bar and is auto-approved for M52 (branch: 2743)
,
Jun 20 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fa264c4c9aa035612932699c793577c341593d0e commit fa264c4c9aa035612932699c793577c341593d0e Author: spqchan <spqchan@chromium.org> Date: Mon Jun 20 16:52:57 2016 [Mac] Fix Toolbar glimpse during fullscreen transition BUG= 618594 , 615810 Review-Url: https://codereview.chromium.org/2078733002 Cr-Commit-Position: refs/heads/master@{#400450} (cherry picked from commit 04542485cce24e64e794c1dfd47d97fa5fbdf20c) Review URL: https://codereview.chromium.org/2080793002 . Cr-Commit-Position: refs/branch-heads/2743@{#400} Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939} [modify] https://crrev.com/fa264c4c9aa035612932699c793577c341593d0e/chrome/browser/ui/cocoa/browser_window_controller.h [modify] https://crrev.com/fa264c4c9aa035612932699c793577c341593d0e/chrome/browser/ui/cocoa/browser_window_controller.mm [modify] https://crrev.com/fa264c4c9aa035612932699c793577c341593d0e/chrome/browser/ui/cocoa/presentation_mode_controller.h [modify] https://crrev.com/fa264c4c9aa035612932699c793577c341593d0e/chrome/browser/ui/cocoa/presentation_mode_controller.mm
,
Jun 20 2016
,
Jun 22 2016
Verified the fix on MAC (10.11.5) for Google Chrome Beta Version - 52.0.2743.49 Screen-recording is attached. TE-Verified labels are added.
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by tkonch...@chromium.org
, Jun 9 2016