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

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

Issue 1990163003: Mac fullscreen low power: Add layout interactions and flag (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add nil check for tests Created 4 years, 7 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/fullscreen_low_power_coordinator.h
diff --git a/chrome/browser/ui/cocoa/fullscreen_low_power_coordinator.h b/chrome/browser/ui/cocoa/fullscreen_low_power_coordinator.h
index 406b5f73ee549ab1f7e608450711be01e0437c60..15c4e546244d70e53242f9a4488527f576a48948 100644
--- a/chrome/browser/ui/cocoa/fullscreen_low_power_coordinator.h
+++ b/chrome/browser/ui/cocoa/fullscreen_low_power_coordinator.h
@@ -29,8 +29,10 @@ class FullscreenLowPowerCoordinatorCocoa
// Interface to BrowserWindowController.
NSWindow* GetFullscreenLowPowerWindow();
- void AddLowPowerModeSuppression();
- void RemoveLowPowerModeSuppression();
+ void SetLayoutParameters(const NSRect& toolbar_frame,
+ const NSRect& infobar_frame,
+ const NSRect& content_frame,
+ const NSRect& download_shelf_frame);
// ui::FullscreenLowPowerCoordinator implementation.
void SetLowPowerLayerValid(bool valid) override;
@@ -55,9 +57,9 @@ class FullscreenLowPowerCoordinatorCocoa
// enter low power mode.
bool low_power_layer_valid_ = false;
- // The balance of calls to Add/RemoveLowPowerModeSuppression. This must be
- // zero to enter low power mode.
- int suppression_count_ = 0;
+ // Set if the NSView hierarchy allows low power mode. Low power mode is only
+ // allowed when nothing but the web contents is on-screen.
+ bool allowed_by_nsview_layout_ = true;
// Updated by EnterOrExitLowPowerModeIfNeeded.
bool in_low_power_mode_ = false;

Powered by Google App Engine
This is Rietveld 408576698