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

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

Issue 2256993002: [Mac] Reworked FullscreenToolbarController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix for erikchen 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_controller_private.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller_private.mm b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
index bb3abfdfbf8acbef7833d8cfc043f48a5f7bfdf8..6bbb6d210f397861d7d4aa664cb132e646486e32 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_private.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
@@ -460,10 +460,8 @@ willPositionSheet:(NSWindow*)sheet
BOOL showDropdown =
!fullscreenForTab && !kioskMode && ([self floatingBarHasFocus]);
- NSView* contentView = [[self window] contentView];
[fullscreenToolbarController_
- setupFullscreenToolbarForContentView:contentView
- showDropdown:showDropdown];
+ setupFullscreenToolbarWithDropdown:showDropdown];
}
- (void)adjustUIForExitingFullscreenAndStopOmniboxSliding {
@@ -610,7 +608,7 @@ willPositionSheet:(NSWindow*)sheet
if (enteringAppKitFullscreen_)
return;
- [self hideOverlayIfPossibleWithAnimation:NO delay:NO];
+ [self hideOverlayIfPossibleWithAnimation:NO];
switch (exclusiveAccessController_->bubble_type()) {
case EXCLUSIVE_ACCESS_BUBBLE_TYPE_NONE:
@@ -855,9 +853,9 @@ willPositionSheet:(NSWindow*)sheet
barVisibilityUpdatesEnabled_ = YES;
if ([barVisibilityLocks_ count])
- [fullscreenToolbarController_ ensureOverlayShownWithAnimation:NO delay:NO];
+ [fullscreenToolbarController_ ensureOverlayShownWithAnimation:NO];
else
- [fullscreenToolbarController_ ensureOverlayHiddenWithAnimation:NO delay:NO];
+ [fullscreenToolbarController_ ensureOverlayHiddenWithAnimation:NO];
}
- (void)disableBarVisibilityUpdates {
@@ -866,14 +864,14 @@ willPositionSheet:(NSWindow*)sheet
return;
barVisibilityUpdatesEnabled_ = NO;
- [fullscreenToolbarController_ cancelAnimationAndTimers];
+ [fullscreenToolbarController_ cancelAnimationAndTimer];
}
-- (void)hideOverlayIfPossibleWithAnimation:(BOOL)animation delay:(BOOL)delay {
+- (void)hideOverlayIfPossibleWithAnimation:(BOOL)animation {
if (!barVisibilityUpdatesEnabled_ || [barVisibilityLocks_ count])
return;
- [fullscreenToolbarController_ ensureOverlayHiddenWithAnimation:animation
- delay:delay];
+
+ [fullscreenToolbarController_ ensureOverlayHiddenWithAnimation:animation];
}
- (CGFloat)toolbarDividerOpacity {
@@ -1011,11 +1009,8 @@ willPositionSheet:(NSWindow*)sheet
[self layoutTabContentArea:output.contentAreaFrame];
- if (!NSIsEmptyRect(output.fullscreenBackingBarFrame)) {
+ if (!NSIsEmptyRect(output.fullscreenBackingBarFrame))
[floatingBarBackingView_ setFrame:output.fullscreenBackingBarFrame];
- [fullscreenToolbarController_
- overlayFrameChanged:output.fullscreenBackingBarFrame];
- }
[findBarCocoaController_
positionFindBarViewAtMaxY:output.findBarMaxY
« no previous file with comments | « chrome/browser/ui/cocoa/browser_window_controller_private.h ('k') | chrome/browser/ui/cocoa/fullscreen_toolbar_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698