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

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

Issue 2505653003: [Mac[ FullscreenToolbarController Refactor (Closed)
Patch Set: nits Created 4 years, 1 month 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 24c805e30895f6e93d0633a6edceabd291757e79..7889ab4bd1bb8accd2ff2eab5e1c2ab2fdcc9411 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_private.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
@@ -33,10 +33,11 @@
#import "chrome/browser/ui/cocoa/floating_bar_backing_view.h"
#import "chrome/browser/ui/cocoa/framed_browser_window.h"
#include "chrome/browser/ui/cocoa/fullscreen_low_power_coordinator.h"
-#import "chrome/browser/ui/cocoa/fullscreen_toolbar_controller.h"
+#import "chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h"
#import "chrome/browser/ui/cocoa/fullscreen_window.h"
#import "chrome/browser/ui/cocoa/infobars/infobar_container_controller.h"
#include "chrome/browser/ui/cocoa/last_active_browser_cocoa.h"
+#include "chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h"
#import "chrome/browser/ui/cocoa/profiles/avatar_button_controller.h"
#import "chrome/browser/ui/cocoa/profiles/avatar_icon_controller.h"
#import "chrome/browser/ui/cocoa/status_bubble_mac.h"
@@ -862,12 +863,12 @@ willPositionSheet:(NSWindow*)sheet
[layout setWindowSize:windowSize];
[layout setInAnyFullscreen:[self isInAnyFullscreenMode]];
- [layout setFullscreenToolbarStyle:fullscreenToolbarController_.get()
- .toolbarStyle];
- [layout
- setFullscreenMenubarOffset:[fullscreenToolbarController_ menubarOffset]];
- [layout setFullscreenToolbarFraction:[fullscreenToolbarController_
- toolbarFraction]];
+
+ FullscreenToolbarLayout fullscreenToolbarLayout =
+ [fullscreenToolbarController_ computeLayout];
+ [layout setFullscreenToolbarStyle:fullscreenToolbarLayout.toolbarStyle];
+ [layout setFullscreenMenubarOffset:fullscreenToolbarLayout.menubarOffset];
+ [layout setFullscreenToolbarFraction:fullscreenToolbarLayout.toolbarFraction];
[layout setHasTabStrip:[self hasTabStrip]];
[layout setFullscreenButtonFrame:[self fullscreenButtonFrame]];

Powered by Google App Engine
This is Rietveld 408576698