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

Unified Diff: chrome/browser/ui/cocoa/browser_window_controller.h

Issue 523723002: mac, fullscreen: Several bug fixes after major refactor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@new_fullscreen2
Patch Set: Created 6 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.h
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.h b/chrome/browser/ui/cocoa/browser_window_controller.h
index 10544a7aa181732acb0206acc9f2c4e0c7df2a28..eac1da5a0ce611fb45e314fbbdcb589406cdc220 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.h
+++ b/chrome/browser/ui/cocoa/browser_window_controller.h
@@ -57,14 +57,6 @@ namespace extensions {
class Command;
}
-namespace fullscreen_mac {
-enum SlidingStyle {
- OMNIBOX_TABS_PRESENT = 0, // Tab strip and omnibox both visible.
- OMNIBOX_PRESENT, // Tab strip hidden.
- OMNIBOX_TABS_HIDDEN, // Tab strip and omnibox both hidden.
-};
-} // namespace fullscreen_mac
-
@interface BrowserWindowController :
TabWindowController<NSUserInterfaceValidations,
BookmarkBarControllerDelegate,
@@ -142,6 +134,10 @@ enum SlidingStyle {
// AppKit fullscreen mode.
BOOL enteringAppKitFullscreen_;
+ // Only adjust the tab strip once while entering fullscreen. See the code
+ // where this is used for a more detailed explanation.
Robert Sesek 2014/09/02 19:59:48 Name the actual method so the reader does not have
erikchen 2014/09/02 20:43:21 Done.
+ BOOL hasAdjustedTabStripWhileEnteringAppKitFullscreen_;
+
// True between |enterImmersiveFullscreen| and |-windowDidEnterFullScreen:|
// to indicate that the window is in the process of transitioning into
// AppKit fullscreen mode.
@@ -160,9 +156,6 @@ enum SlidingStyle {
// The proportion of the floating bar which is shown (in presentation mode).
CGFloat floatingBarShownFraction_;
- // Whether the omnibox is hidden in fullscreen.
- fullscreen_mac::SlidingStyle fullscreenStyle_;
-
// Various UI elements/events may want to ensure that the floating bar is
// visible (in presentation mode), e.g., because of where the mouse is or
// where keyboard focus is. Whenever an object requires bar visibility, it has
@@ -531,11 +524,6 @@ enum SlidingStyle {
// resolution.
- (void)resizeFullscreenWindow;
-// Gets or sets the fraction of the floating bar (presentation mode overlay)
-// that is shown. 0 is completely hidden, 1 is fully shown.
-- (CGFloat)floatingBarShownFraction;
-- (void)setFloatingBarShownFraction:(CGFloat)fraction;
-
// Query/lock/release the requirement that the tab strip/toolbar/attached
// bookmark bar bar cluster is visible (e.g., when one of its elements has
// focus). This is required for the floating bar in presentation mode, but

Powered by Google App Engine
This is Rietveld 408576698