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

ui/views/widget/native_widget_aura_unittest.cc

Index: ui/views/widget/native_widget_aura_unittest.cc
diff --git a/ui/views/widget/native_widget_aura_unittest.cc b/ui/views/widget/native_widget_aura_unittest.cc
index 910718b2863fcd30cfdbaadd60a377f2a249258f..166b651351bfea33bb6fa308b5fb34c2fdcc44c8 100644
--- a/ui/views/widget/native_widget_aura_unittest.cc
+++ b/ui/views/widget/native_widget_aura_unittest.cc
@@ -100,6 +100,22 @@ TEST_F(NativeWidgetAuraTest, CenterWindowSmallParentNotAtOrigin) {
   widget->CloseNow();
 }
 
+TEST_F(NativeWidgetAuraTest, CreateMinimized) {
+  Widget::InitParams params(Widget::InitParams::TYPE_WINDOW);
+  params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  params.parent = NULL;
+  params.context = root_window();
+  params.show_state = ui::SHOW_STATE_MINIMIZED;
+  params.bounds.SetRect(0, 0, 1024, 800);
+  scoped_ptr<Widget> widget(new Widget());
+  widget->Init(params);
+  widget->Show();
+  widget->Minimize();

[joone, 2015/07/01 00:05:42]

Now, this test case is almost like the change in ShowWithWindowState().

[sadrul, 2015/07/01 03:50:56]

Isn't the purpose of this CL to make sure Show() does the right thing, and an
explicit call to Minimize() isn't necessary? Isn't that the change you are
making in tabs_api.cc too? Why do we need to call Minimize() here, then?

[joone, 2015/07/01 22:50:15]

Yes, Minimized() should not be necessary, but the show_state can be changed to
SHOW_STATE_DEFAULT in Widget::Show() by passing  saved_show_state_ to
ShowWithWindowState(). I think Widget::Show() should consider
SHOW_STATE_MINIMIZED state like SHOW_STATE_FULLSCREEN state.

+
+  EXPECT_TRUE(widget->IsMinimized());
+  widget->CloseNow();
+}
+
 class TestLayoutManagerBase : public aura::LayoutManager {
  public:
   TestLayoutManagerBase() {}