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

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

Issue 2040253003: Mac low power video: Support child windows, sheets, and transitions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/framed_browser_window.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1564f0c8bc974b6852ae0f5d895d4bb0a7a869e1..b6ce3acc059d881b2597ef04cee0481b18c728ad 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_private.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
@@ -761,9 +761,15 @@ - (void)windowDidEnterFullScreen:(NSNotification*)notification {
[self showFullscreenExitBubbleIfNecessary];
browser_->WindowFullscreenStateChanged();
+
+ if (fullscreenLowPowerCoordinator_)
+ fullscreenLowPowerCoordinator_->SetInFullscreenTransition(false);
spqchan 2016/06/07 22:30:32 Why aren't we setting it to true in windowWillEnte
ccameron 2016/06/07 23:12:50 Good point -- updated.
}
- (void)windowWillExitFullScreen:(NSNotification*)notification {
+ if (fullscreenLowPowerCoordinator_)
+ fullscreenLowPowerCoordinator_->SetInFullscreenTransition(true);
+
if (notification) // For System Fullscreen when non-nil.
[self registerForContentViewResizeNotifications];
exitingAppKitFullscreen_ = YES;
@@ -1268,4 +1274,19 @@ - (BOOL)isFullscreenForTabContentOrExtension {
controller->IsExtensionFullscreenOrPending();
}
+- (void)windowWillBeginSheet:(NSNotification*)notification {
+ if (fullscreenLowPowerCoordinator_)
+ fullscreenLowPowerCoordinator_->SetHasActiveSheet(true);
+}
+
+- (void)windowDidEndSheet:(NSNotification*)notification {
+ if (fullscreenLowPowerCoordinator_)
+ fullscreenLowPowerCoordinator_->SetHasActiveSheet(false);
+}
+
+- (void)childWindowsDidChange {
+ if (fullscreenLowPowerCoordinator_)
+ fullscreenLowPowerCoordinator_->ChildWindowsChanged();
+}
+
@end // @implementation BrowserWindowController(Private)
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/framed_browser_window.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698