Restoring a session should restore window minimization state on Windows

views/views_delegate.h

Index: views/views_delegate.h
diff --git a/views/views_delegate.h b/views/views_delegate.h
index 13baff03a640a0b6680cc83b21d33880decd9dd7..1f3d98ac5ab947692fed7a0be0e03e9aecc9fd6f 100644
--- a/views/views_delegate.h
+++ b/views/views_delegate.h
@@ -12,6 +12,7 @@
 #endif
 
 #include "ui/base/accessibility/accessibility_types.h"
+#include "ui/base/ui_base_types.h"
 #include "views/views_export.h"
 
 namespace gfx {
@@ -46,22 +47,23 @@ class VIEWS_EXPORT ViewsDelegate {
   // TODO(beng): perhaps this should be a Widget.
   virtual View* GetDefaultParentView() = 0;
 
-  // Saves the position, size and maximized state for the window with the
+  // Saves the position, size and "show" state for the window with the
   // specified name.
   virtual void SaveWindowPlacement(const Widget* widget,
                                    const std::wstring& window_name,
                                    const gfx::Rect& bounds,
-                                   bool maximized) = 0;
+                                   ui::WindowShowState show_state) = 0;
 
   // Retrieves the saved position and size for the window with the specified
   // name.
   virtual bool GetSavedWindowBounds(const std::wstring& window_name,
                                     gfx::Rect* bounds) const = 0;
 
-  // Retrieves the saved maximized state for the window with the specified
+  // Retrieves the saved "show" state for the window with the specified
   // name.
-  virtual bool GetSavedMaximizedState(const std::wstring& window_name,
-                                      bool* maximized) const = 0;
+  virtual bool GetSavedWindowShowState(

[sky, 2011/08/26 16:26:20]

Since you're here, is it possible to combine GetSavedWindowBounds and
GetSavedWindowShowState into one method that takes the gfx::Rect* and
windowshowstate* ? I don't know why we ended up with two.

[dhollowa, 2011/08/26 22:13:50]

Done.

I've gone with |GetSavedWindowPlacement| throughout.  This applies to the
ViewsDelegate hierarchy as well as the WidgetDelegate hierarchy.

I've also propagated the "Placement" terminology into
|NativeWidget::GetWindowPlacement| (it was
|NativeWidget::GetWindowBoundsAndShowState|).

There are further opportunities in |WindowSizer| and |Browser| but I stopped
short because the use-patterns with those are a touch complex.  A follow-up
patch possibly.

+      const std::wstring& window_name,
+      ui::WindowShowState* show_state) const = 0;
 
   virtual void NotifyAccessibilityEvent(
       views::View* view, ui::AccessibilityTypes::Event event_type) = 0;