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

Unified Diff: chrome/browser/automation/automation_provider_observers.h

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/automation/automation_provider_observers.h
diff --git a/chrome/browser/automation/automation_provider_observers.h b/chrome/browser/automation/automation_provider_observers.h
index 073082d353e06a2e9c83d5bda7401e9518da9c17..a4f8a2f2e0107eca748a589460d628c2a040655c 100644
--- a/chrome/browser/automation/automation_provider_observers.h
+++ b/chrome/browser/automation/automation_provider_observers.h
@@ -1857,4 +1857,25 @@ class ExtensionPopupObserver : public content::NotificationObserver {
DISALLOW_COPY_AND_ASSIGN(ExtensionPopupObserver);
};
+// Allows the automation provider to wait for a WINDOW_MAXIMIZED notification.
+class WindowMaximizedObserver : public content::NotificationObserver {
+ public:
+ WindowMaximizedObserver(AutomationProvider* automation,
+ IPC::Message* reply_message);
kkania 2012/05/30 01:03:58 when defining or prototyping a function, the args,
zori 2012/05/31 23:38:22 Done.
+ virtual ~WindowMaximizedObserver();
+
+ virtual void Observe(int type,
kkania 2012/05/30 01:03:58 It's typical style to put: // Overridden from Noti
zori 2012/05/31 23:38:22 Done.
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details);
kkania 2012/05/30 01:03:58 add OVERRIDE macro
zori 2012/05/31 23:38:22 Done.
+
+ private:
+ content::NotificationRegistrar registrar_;
+ base::WeakPtr<AutomationProvider> automation_;
+ scoped_ptr<IPC::Message> reply_message_;
+
+ DISALLOW_COPY_AND_ASSIGN(WindowMaximizedObserver);
+};
+
+
+
#endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_

Powered by Google App Engine
This is Rietveld 408576698