Support maximize window command.

chrome/browser/ui/gtk/browser_window_gtk.cc

Index: chrome/browser/ui/gtk/browser_window_gtk.cc
diff --git a/chrome/browser/ui/gtk/browser_window_gtk.cc b/chrome/browser/ui/gtk/browser_window_gtk.cc
index 643bc5baa05bcb016ba7e4d7e3b8c1f1e26e7101..cdb0dacc3bd30e533146c22bc5d05581420fa563 100644
--- a/chrome/browser/ui/gtk/browser_window_gtk.cc
+++ b/chrome/browser/ui/gtk/browser_window_gtk.cc
@@ -1603,6 +1603,13 @@ gboolean BrowserWindowGtk::OnWindowState(GtkWidget* sender,
     }
   }
 
+  if (event->changed_mask & GDK_WINDOW_STATE_MAXIMIZED) {

[Ben Goodger (Google), 2012/06/01 17:10:11]

is this already supported on other platforms?

can we avoid using notifications?

[zori, 2012/06/02 01:20:54]

Since other platforms execute the maximize operation synchronously, the testing
automation provider in chrome is able to send a 'success' JSON reply right after
invoking the maximize() command on the browser window. On Linux, however, this
happens asynchronously, so we should wait for the gtk event indicating
completion of maximization. Could you, perhaps, suggest a better approach than
using the Notification Service?

+    content::NotificationService::current()->Notify(
+        chrome::NOTIFICATION_BROWSER_WINDOW_MAXIMIZED,
+        content::Source<BrowserWindowGtk>(this),

[kkania, 2012/06/01 04:14:03]

Just use BrowserWindow

[zori, 2012/06/02 01:20:54]

Done.

+        content::NotificationService::NoDetails());
+  }
+
   titlebar_->UpdateCustomFrame(UseCustomFrame() && !IsFullscreen());
   UpdateWindowShape(bounds_.width(), bounds_.height());
   SaveWindowPosition();