Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4974)

Unified Diff: chrome/browser/ui/cocoa/browser_window_layout.mm

Issue 2272783002: [Mac] Fix for fullscreen toolbar (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/browser_window_layout.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_layout.mm b/chrome/browser/ui/cocoa/browser_window_layout.mm
index 23ca75e9c224bc8040a7022e452b7cdde194d18a..bcb64898702ffa3d854b7e5ca3effb1eada7f25c 100644
--- a/chrome/browser/ui/cocoa/browser_window_layout.mm
+++ b/chrome/browser/ui/cocoa/browser_window_layout.mm
@@ -168,17 +168,8 @@ const CGFloat kLocationBarRightOffset = 35;
CGFloat yOffset = 0;
if (parameters_.inAnyFullscreen) {
yOffset += parameters_.menubarOffset;
- switch (parameters_.slidingStyle) {
- case fullscreen_mac::OMNIBOX_TABS_PRESENT:
- break;
- case fullscreen_mac::OMNIBOX_TABS_NONE:
- case fullscreen_mac::OMNIBOX_TABS_HIDDEN:
- // In presentation mode, |yOffset| accounts for the sliding position of
- // the floating bar and the extra offset needed to dodge the menu bar.
- yOffset += std::floor((1 - parameters_.toolbarFraction) *
- [self fullscreenBackingBarHeight]);
- break;
- }
+ yOffset += std::floor((1 - parameters_.toolbarFraction) *
+ [self fullscreenBackingBarHeight]);
}
fullscreenYOffset_ = yOffset;
}

Powered by Google App Engine
This is Rietveld 408576698