Make the minimize state work when creating a chrome window in CrOS

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 bf113e6e119de7bcb3ab99b2f21a1d185eba9ef7..d1dd97ea9d7e72d68af596f3c45e10acbc6464a0 100644
--- a/ui/views/widget/native_widget_aura.cc
+++ b/ui/views/widget/native_widget_aura.cc
@@ -484,8 +484,10 @@ void NativeWidgetAura::ShowWithWindowState(ui::WindowShowState state) {
   if (!window_)
     return;
 
-  if (state == ui::SHOW_STATE_MAXIMIZED || state == ui::SHOW_STATE_FULLSCREEN)
-    window_->SetProperty(aura::client::kShowStateKey, state);
+  window_->SetProperty(aura::client::kShowStateKey, state);
+  if (state == ui::SHOW_STATE_MINIMIZED)
+    return;
+

[sadrul, 2015/05/18 19:51:42]

Why? This looks like a change in behaviour.

[joone, 2015/05/18 20:22:43]

The minimized state doesn't work when we running the blow code.

[sadrul, 2015/05/18 20:29:27]

Why? We currently do run the code below for MINIMIZED, right? Why can't we do
that if we set the property?

[oshima, 2015/05/18 20:48:06]

Showing (or activating) minimized window will unminimize the window, and that's
probably why. That's being said, allowing MINIMINIZED here may sounds a
contradiction because a minimized window isn't shown. Maybe caller should just
call minimized?

   window_->Show();
   if (delegate_->CanActivate()) {
     if (state != ui::SHOW_STATE_INACTIVE)