top bar in chrome does not return to top of screen in full screen view
Reported by
sethjben...@gmail.com,
Aug 13 2016
|
|||
Issue description
Chrome Version : 52.0.2743.116
OS Version: OS X 10.10.5
URLs (if applicable) :
Other browsers tested:
Add OK or FAIL after other browsers where you have tested this issue:
Safari 5: OK
Firefox 4.x:
IE 7/8/9:
What steps will reproduce the problem?
1. Open chrome and set to full screen (green button in upper left panel)
2. Bring mouse cursor to top of screen to show mac menu bar
3. With cursor still at the top of the screen (ie, with mac menu bar still showing), swipe back to view desktop (usually, swiping with three fingers on trackpad)
4. Move the cursor away from top of screen, and swipe back to view chrome
What is the expected result?
The menu bar should disappear, and the top bar of chrome (containing open tabs, etc) should move back to the top of the screen.
What happens instead of that?
The menu bar disappears, but the top bar of chrome does not properly move back to the top, leaving a blank empty space in between and obscuring the top of the page being displayed. This goes away if the cursor is moved back to the top of the screen, but it is a nuisance to have to do this.
Please provide any additional information below. Attach a screenshot if
possible.
The blank space is visible at the top of the attached screenshot.
UserAgentString: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36
,
Aug 15 2016
Oof, seems like a pretty serious bug.
,
Aug 19 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f commit cccd612ee508dd2ee75f5d70e46b33bb17a7b88f Author: spqchan <spqchan@chromium.org> Date: Fri Aug 19 01:46:46 2016 [Mac] Reworked FullscreenToolbarController Removed mouse tracking for the menubar since it's unreliable for multiple monitors and SplitScreen. Instead, use the menubar Carbon events to update the toolbar. Removed the animation delays because they were mostly unused. When they actually were used, they would cause bugs and janky movements. Cleaned up dead code and comments BUG= 634981 , 637976 , 637514 Review-Url: https://codereview.chromium.org/2256993002 Cr-Commit-Position: refs/heads/master@{#413018} [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_observer_cocoa.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/bookmarks/bookmark_button.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/bookmarks/bookmark_editor_base_controller.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/browser_window_controller.h [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/browser_window_controller.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/browser_window_controller_private.h [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/browser_window_controller_private.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/fullscreen_toolbar_controller.h [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/fullscreen_toolbar_controller.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/global_error_bubble_controller.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_editor.mm [modify] https://crrev.com/cccd612ee508dd2ee75f5d70e46b33bb17a7b88f/chrome/browser/ui/cocoa/profiles/avatar_base_controller.mm
,
May 22 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by meh...@chromium.org
, Aug 13 2016Components: UI>Browser>FullScreen