Combined window positioning and show state determiniation in the WindowSizer

chrome/browser/ui/browser_window_state.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/browser_window_state.h"
 
 #include "base/command_line.h"
 #include "base/string_number_conversions.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/prefs/pref_service.h"
@@ skipping 62 matching lines @@
   // Save to the session storage service, used when reloading a past session.
   // Note that we don't want to be the ones who cause lazy initialization of
   // the session service. This function gets called during initial window
   // showing, and we don't want to bring in the session service this early.
   SessionService* session_service =
       SessionServiceFactory::GetForProfileIfExisting(browser->profile());
   if (session_service)
     session_service->SetWindowBounds(browser->session_id(), bounds, show_state);
 }
 
-gfx::Rect GetSavedWindowBounds(const Browser* browser) {
+gfx::Rect GetSavedWindowBoundsAndShowState(const Browser* browser,
+                                           ui::WindowShowState& show_state) {
   gfx::Rect restored_bounds = browser->override_bounds();
-  WindowSizer::GetBrowserWindowBounds(browser->app_name(),
-                                      restored_bounds,
-                                      browser,
-                                      &restored_bounds);
+  WindowSizer::GetBrowserWindowBoundsAndShowState(browser->app_name(),
+                                                  restored_bounds,
+                                                  browser,
+                                                  &restored_bounds,
+                                                  show_state);
 
   const CommandLine& parsed_command_line = *CommandLine::ForCurrentProcess();
   bool record_mode = parsed_command_line.HasSwitch(switches::kRecordMode);
   bool playback_mode = parsed_command_line.HasSwitch(switches::kPlaybackMode);
   if (record_mode || playback_mode) {
     // In playback/record mode we always fix the size of the browser and
     // move it to (0,0).  The reason for this is two reasons:  First we want
     // resize/moves in the playback to still work, and Second we want
     // playbacks to work (as much as possible) on machines w/ different
     // screen sizes.
@@ skipping 12 matching lines @@
     std::string str =
         parsed_command_line.GetSwitchValueASCII(switches::kWindowPosition);
     int x, y;
     if (ParseCommaSeparatedIntegers(str, &x, &y))
       restored_bounds.set_origin(gfx::Point(x, y));
   }
 
   return restored_bounds;
 }
 
-ui::WindowShowState GetSavedWindowShowState(const Browser* browser) {
-  // Only tabbed browsers use the command line or preference state, with the
-  // exception of devtools.
-  bool show_state = !browser->is_type_tabbed() && !browser->is_devtools();
-
-#if defined(USE_AURA)
-  // Apps save state on aura.
-  show_state &= !browser->is_app();
-#endif
-
-  if (show_state)
-    return browser->initial_show_state();
-
-  if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kStartMaximized))
-    return ui::SHOW_STATE_MAXIMIZED;
-
-  if (browser->initial_show_state() != ui::SHOW_STATE_DEFAULT)
-    return browser->initial_show_state();
-
-  const DictionaryValue* window_pref = browser->profile()->GetPrefs()->
-      GetDictionary(GetWindowPlacementKey(browser).c_str());
-  bool maximized = false;
-  window_pref->GetBoolean("maximized", &maximized);
-
-  return maximized ? ui::SHOW_STATE_MAXIMIZED : ui::SHOW_STATE_DEFAULT;
-}
-
 }  // namespace chrome