Adding experimental maximize mode (behind a flag)

chrome/browser/sessions/session_restore.cc

Index: chrome/browser/sessions/session_restore.cc
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc
index 8e65c18c5e1e1843cc3b0cc57464e30d535c1de9..45264b9227fd8ef24e22b14e741c479a46537306 100644
--- a/chrome/browser/sessions/session_restore.cc
+++ b/chrome/browser/sessions/session_restore.cc
@@ -60,6 +60,7 @@
 #endif
 
 #if defined(USE_ASH)
+#include "ash/shell.h"
 #include "ash/wm/window_util.h"
 #endif
 using content::NavigationController;
@@ -1035,7 +1036,12 @@ class SessionRestoreImpl : public content::NotificationObserver {
     Browser::CreateParams params(type, profile_, host_desktop_type_);
     params.app_name = app_name;
     params.initial_bounds = bounds;
-    params.initial_show_state = show_state;
+#if defined(USE_ASH)
+    if (ash::Shell::IsForcedMaximizeMode())
+      params.initial_show_state = ui::SHOW_STATE_MAXIMIZED;
+    else
+#endif

[James Cook, 2013/04/22 22:15:46]

#ifdefs that mess with indentation I find hard to read. How about:
#if defined(USE_ASH)
  if (ash::Shell:IsForceMaximizeMode())
    show_state = ui::SHOW_STATE_MAXIMIZED;
#endif

[Mr4D (OOO till 08-26), 2013/04/23 04:41:28]

I tried to avoid that. However since you want it: Done.

+      params.initial_show_state = show_state;
     params.is_session_restore = true;
     return new Browser(params);
   }