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

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

Issue 10388251: Support maximize window command. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: |BrowserWindow| (gtk and cocoa) will notify when a window is maximized. Created 8 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/browser_window_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
index 344aadceab25e5db9840f7371d2b7ce335413704..209c6deede97b5536f41a13866a4b2603ce664d4 100644
--- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
+++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
@@ -38,6 +38,7 @@
#include "chrome/browser/ui/webui/task_manager/task_manager_dialog.h"
#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/pref_names.h"
+#include "content/public/browser/notification_service.h"
#include "content/public/browser/notification_source.h"
#include "content/public/browser/native_web_keyboard_event.h"
#include "content/public/browser/notification_details.h"
@@ -276,8 +277,13 @@ bool BrowserWindowCocoa::IsMinimized() const {
void BrowserWindowCocoa::Maximize() {
// Zoom toggles so only call if not already maximized.
- if (!IsMaximized())
+ if (!IsMaximized()) {
[window() zoom:controller_];
+ content::NotificationService::current()->Notify(
kkania 2012/05/30 01:03:58 I don't think we need to use a notification here i
zori 2012/05/31 23:38:22 Done.
+ chrome::NOTIFICATION_WINDOW_MAXIMIZED,
+ content::Source<NSWindow>(window()),
+ content::NotificationService::NoDetails());
+ }
}
void BrowserWindowCocoa::Minimize() {

Powered by Google App Engine
This is Rietveld 408576698