Set minimize property and re-enable tests on Aura

ui/views/widget/native_widget_aura.cc

Index: ui/views/widget/native_widget_aura.cc
diff --git a/ui/views/widget/native_widget_aura.cc b/ui/views/widget/native_widget_aura.cc
index 098eeaed904ba314b516efa0355ac5cc8ee5e6b7..4e94a2314a84e584d07807a2ad73cd2c83da2170 100644
--- a/ui/views/widget/native_widget_aura.cc
+++ b/ui/views/widget/native_widget_aura.cc
@@ -415,6 +415,8 @@ gfx::Rect NativeWidgetAura::GetRestoredBounds() const {
 }
 
 void NativeWidgetAura::SetBounds(const gfx::Rect& bounds) {
+  if (GetRestoreBounds(window_))

[Ben Goodger (Google), 2012/02/23 18:27:37]

Why is this conditional on restore bounds already having been set?

[stevenjb, 2012/02/23 18:41:02]

I didn't want to change the behavior if kRestoreBoundsKey has never been set. 
On the other hand, currently if we call GetRestoreBounds() without minimizing or
maximizing first it will return NULL, so maybe we do want to set it always?

[oshima, 2012/02/23 18:46:20]

If window is in normal state, there is nothing to restore to. I believe this
should be done in ash if not already done so.

[stevenjb, 2012/02/23 18:54:23]

Actually, I was wrong, I was looking at the wrong function.
GetRestoredBounds() does check to see if the property exists, and if it does not
then it returns window_->bounds().

So, this is just a question of whether or not we want to always set the
property.

[oshima, 2012/02/24 00:29:39]

Restore bounds make sense only when the window is not in normal state,
so I don't think we want to set it always. If we want/need to update the
restore bounds to new bounds while the window is in max/min/full state,
it should be done in ash rather than in views.
(NWA has "delete GetRestoreBounds(window)" which is a workaround to fix memory
leak.
 This will be fixed by benrg (not beng:))

+    SetRestoreBounds(window_, bounds);
   window_->SetBounds(bounds);
 }
 
@@ -516,8 +518,9 @@ void NativeWidgetAura::Maximize() {
 }
 
 void NativeWidgetAura::Minimize() {
-  // No minimized window for aura. crbug.com/104571.
-  NOTREACHED();
+  // Note: This currently does not do anything except set the property,
+  // see crbug.com/104571.
+  window_->SetProperty(aura::client::kShowStateKey, ui::SHOW_STATE_MINIMIZED);
 }
 
 bool NativeWidgetAura::IsMaximized() const {