New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 637514 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

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



 
bug.png
137 KB View Download

Comment 1 by meh...@chromium.org, Aug 13 2016

Cc: spqc...@chromium.org
Components: UI>Browser>FullScreen
Cc: -spqc...@chromium.org
Owner: spqc...@chromium.org
Status: Assigned (was: Unconfirmed)
Oof, seems like a pretty serious bug.
Project Member

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

Status: Fixed (was: Assigned)

Sign in to add a comment