Re-do the way browser windows are shown:...

chrome/browser/browser.h

 // Copyright (c) 2006-2008 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.
 
 #ifndef CHROME_BROWSER_BROWSER_H_
 #define CHROME_BROWSER_BROWSER_H_
 
 #include "chrome/browser/controller.h"
 #include "chrome/browser/shell_dialogs.h"
 #include "chrome/browser/browser_type.h"
+#include "chrome/browser/browser_window.h"
 #include "chrome/browser/session_id.h"
 #include "chrome/browser/tab_contents_delegate.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 #include "chrome/browser/toolbar_model.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/pref_member.h"
 
 class BrowserIdleTimer;
 class BrowserWindow;
 class DebuggerWindow;
 class GoButton;
 class LocationBarView;
 class PrefService;
 class Profile;
 class StatusBubble;
 struct TabNavigation;
-class WebContents;
 class WebApp;
 
 class Browser : public TabStripModelDelegate,
                 public TabStripModelObserver,
                 public TabContentsDelegate,
                 public CommandHandler,
                 public NotificationObserver,
                 public SelectFileDialog::Listener {
  public:
   // Constructors, Creation, Showing //////////////////////////////////////////
 
-  // Creates a new browser with the given bounds. If the bounds are empty, the
-  // system will try to find a saved size from a previous session, if none
-  // exists, the operating system will be allowed to size the window.
-  // |type| defines the kind of browser to create.
-  //
-  // Creating a browser does NOT show the window. You must manually call Show()
-  // to display the window.
-  Browser(const gfx::Rect& initial_bounds,
-          int show_command,
-          Profile* profile,
-          BrowserType::Type browser_type,
-          const std::wstring& app_name);
+  // Creates a new browser of the given |type| and for the given |profile|. The
+  // Browser has a NULL window after its construction, CreateBrowserWindow must
+  // be called after configuration for window() to be valid.
+  Browser(BrowserType::Type type, Profile* profile);
   ~Browser();
 
-  // Shows the browser window. It is initially created hidden. It will be shown
-  // with the show command passed to the constructor, or possibly another state
-  // if it was overridden in the preferences.
-  //
-  // Ideally, this function is called after everything in the window is
-  // initialized so that we do not have to repaint again.
-  void Show();
+  // Creates a normal tabbed browser with the specified profile. The Browser's
+  // window is created by this function call.
+  static Browser* Create(Profile* profile);
+
+  // Like Create, but creates a tabstrip-less popup window.
+  static Browser* CreateForPopup(Profile* profile);
+
+  // Like Create, but creates a tabstrip-less and toolbar-less "app" window for
+  // the specified app.
+  static Browser* CreateForApp(const std::wstring& app_name, Profile* profile);
+
+  // Set overrides for the initial window bounds and maximized state.
+  void set_override_bounds(const gfx::Rect& bounds) {
+    override_bounds_ = bounds;
+  }
+  void set_override_maximized(bool maximized) {
+    override_maximized_ = maximized;
+  }
+
+  // Creates the Browser Window. Prefer to use the static helpers above where
+  // possible. This does not show the window. You need to call window()->Show()
+  // to show it.
+  void CreateBrowserWindow();
 
   // Accessors ////////////////////////////////////////////////////////////////
 
   BrowserType::Type type() const { return type_; }
   Profile* profile() const { return profile_; }
   BrowserWindow* window() const { return window_; }
   ToolbarModel* toolbar_model() { return &toolbar_model_; }
   const SessionID& session_id() const { return session_id_; }
   CommandController* controller() { return &controller_; }
 
   // Browser Creation Helpers /////////////////////////////////////////////////
 
-  // Opens a new browser window for the specified |profile|, shown according to
-  // |show_command|
-  static void OpenNewBrowserWindow(Profile* profile, int show_command);
+  // Opens a new window with the default blank tab.
+  static void OpenEmptyWindow(Profile* profile);
 
   // Opens the specified URL in a new browser window in an incognito session.
   // If there is already an existing active incognito session for the specified
   // |profile|, that session is re-used.
   static void OpenURLOffTheRecord(Profile* profile, const GURL& url);
 
   // Opens the a new application window for the specified WebApp.
-  static void OpenWebApplication(Profile* profile,
-                                 WebApp* app,
-                                 int show_command);
+  static void OpenWebApplication(Profile* profile, WebApp* app);
 
   // Command API //////////////////////////////////////////////////////////////
 
   // Please fix the incestuous nest that is */controller.h and eliminate the
   // need for this retarded hack.
   bool SupportsCommand(int id) const;
   bool IsCommandEnabled(int id) const;
 
   // DEPRECATED DEPRECATED DEPRECATED /////////////////////////////////////////
 
   // Returns the HWND of the top-level system window for this Browser.
   HWND GetTopLevelHWND() const;
 
   // State Storage and Retrieval for UI ///////////////////////////////////////
 
   // Save and restore the window position.
-  void SaveWindowPosition(const gfx::Rect& bounds, bool maximized);
-  void RestoreWindowPosition(gfx::Rect* bounds, bool* maximized);
+  void SaveWindowPlacement(const gfx::Rect& bounds, bool maximized);
+  gfx::Rect GetSavedWindowBounds() const;
+  bool GetSavedMaximizedState() const;
 
   // Gets the FavIcon of the page in the selected tab.
   SkBitmap GetCurrentPageIcon() const;
 
   // Gets the title of the page in the selected tab.
   std::wstring GetCurrentPageTitle() const;
 
   // Prepares a title string for display (removes embedded newlines, etc).
   static void FormatTitleForDisplay(std::wstring* title);
 
@@ skipping 168 matching lines @@
 
   // Interface implementations ////////////////////////////////////////////////
 
   // Overridden from CommandHandler:
   virtual bool GetContextualLabel(int id, std::wstring* out) const {
     return false;
   }
   virtual void ExecuteCommand(int id);
 
   // Overridden from TabStripModelDelegate:
+  virtual GURL GetBlankTabURL() const;
   virtual void CreateNewStripWithContents(TabContents* detached_contents,
                                           const gfx::Point& drop_point);
   virtual int GetDragActions() const;
   // Construct a TabContents for a given URL, profile and transition type.
   // If instance is not null, its process will be used to render the tab.
-  // TODO(beng): remove this from TabStripDelegate, it's only used by
-  //             TabStripModel::AddBlankTab*, which should really live here
-  //             on Browser.
   virtual TabContents* CreateTabContentsForURL(
       const GURL& url,
       const GURL& referrer,
       Profile* profile,
       PageTransition::Type transition,
       bool defer_load,
       SiteInstance* instance) const;
   virtual bool CanDuplicateContentsAt(int index);
   virtual void DuplicateContentsAt(int index);
   virtual void ValidateLoadingAnimations();
@@ skipping 183 matching lines @@
 
   // This Browser's type.
   BrowserType::Type type_;
 
   // This Browser's profile.
   Profile* profile_;
 
   // This Browser's window.
   BrowserWindow* window_;
 
-  // Controls how the window will appear when Show() is called. This is one
-  // of the SW_* constants passed to ShowWindow, and will be initialized in the
-  // constructor.
-  //
-  // After the first call to Show() succeeds, this is set to -1, indicating that
-  // subsequent calls to Show() should be ignored.
-  // TODO(beng): This should be removed (http://crbug.com/3557) and put into
-  //             BrowserView, or some more likely place.
-  int initial_show_command_;
-
   // This Browser's TabStripModel.
   TabStripModel tabstrip_model_;
 
   // The Controller that updates all browser commands.
   CommandController controller_;
 
   // An optional application name which is used to retrieve and save window
   // positions.
   std::wstring app_name_;
 
@@ skipping 46 matching lines @@
   // Tracks tabs that need there unload event fired before we can
   // close the browser. Only gets populated when we try to close the browser.
   UnloadListenerVector tabs_needing_unload_fired_;
 
   // Whether we are processing the beforeunload and unload events of each tab
   // in preparation for closing the browser.
   bool is_attempting_to_close_browser_;
 
   /////////////////////////////////////////////////////////////////////////////
 
+  // Override values for the bounds of the window and its maximized state.
+  // These are supplied by callers that don't want to use the default values.
+  // The default values are typically loaded from local state (last session),
+  // obtained from the last window of the same type, or obtained from the
+  // shell shortcut's startup info.
+  gfx::Rect override_bounds_;
+  bool override_maximized_;
+
   // The following factory is used to close the frame at a later time.
   ScopedRunnableMethodFactory<Browser> method_factory_;
 
   // Debugger Window, created lazily
   scoped_refptr<DebuggerWindow> debugger_window_;
 
   // Dialog box used for opening and saving files.
   scoped_refptr<SelectFileDialog> select_file_dialog_;
 
   // The browser idle task helps cleanup unused memory resources when idle.
   scoped_ptr<BrowserIdleTimer> idle_task_;
 
   // Keep track of the encoding auto detect pref.
   BooleanPrefMember encoding_auto_detect_;
 
   DISALLOW_COPY_AND_ASSIGN(Browser);
 };
 
 #endif  // CHROME_BROWSER_BROWSER_H_