Fix hiding behavior when creating app windows.

chrome/browser/apps/app_window_interactive_uitest.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/apps/app_browsertest_util.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/ui/browser_iterator.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/test/base/interactive_test_utils.h"
 #include "extensions/browser/app_window/native_app_window.h"
@@ skipping 443 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(AppWindowInteractiveTest, MAYBE_TestShow) {
   ASSERT_TRUE(RunAppWindowInteractiveTest("testShow")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(AppWindowInteractiveTest, TestDrawAttention) {
   ASSERT_TRUE(RunAppWindowInteractiveTest("testDrawAttention")) << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(AppWindowInteractiveTest, TestCreateHidden) {
+  // Created hidden both times.
+  {
+    ExtensionTestMessageListener launched_listener("Launched", true);
+    LoadAndLaunchPlatformApp("hidden_with_id", &launched_listener);
+    EXPECT_TRUE(launched_listener.WaitUntilSatisfied());
+    ExtensionTestMessageListener create_listener_1("Launched", true);
+    launched_listener.Reply("createHidden");
+    EXPECT_TRUE(create_listener_1.WaitUntilSatisfied());
+    AppWindow* app_window = GetFirstAppWindow();
+    EXPECT_TRUE(app_window->is_hidden());
+    ExtensionTestMessageListener create_listener_2("Launched", false);
+    create_listener_1.Reply("createHidden");
+    EXPECT_TRUE(create_listener_2.WaitUntilSatisfied());
+    EXPECT_TRUE(app_window->is_hidden());
+    app_window->GetBaseWindow()->Close();
+  }
+
+  // Created hidden, then visible. The second create should show the window.
+  {
+    ExtensionTestMessageListener launched_listener("Launched", true);
+    LoadAndLaunchPlatformApp("hidden_with_id", &launched_listener);
+    EXPECT_TRUE(launched_listener.WaitUntilSatisfied());
+    ExtensionTestMessageListener create_listener_1("Launched", true);
+    launched_listener.Reply("createHidden");
+    EXPECT_TRUE(create_listener_1.WaitUntilSatisfied());
+    AppWindow* app_window = GetFirstAppWindow();
+    EXPECT_TRUE(app_window->is_hidden());
+    ExtensionTestMessageListener create_listener_2("Launched", false);
+    create_listener_1.Reply("createVisible");
+    EXPECT_TRUE(create_listener_2.WaitUntilSatisfied());
+    EXPECT_FALSE(app_window->is_hidden());
+    app_window->GetBaseWindow()->Close();
+  }
+}
+
 // Only Linux and Windows use keep-alive to determine when to shut down.
 #if defined(OS_LINUX) || defined(OS_WIN)
 
 // In general, hidden windows should not keep Chrome alive. The exception is
 // when windows are created hidden, we allow the app some time to show the
 // the window.
 class AppWindowHiddenKeepAliveTest : public extensions::PlatformAppBrowserTest {
  protected:
   AppWindowHiddenKeepAliveTest() {}
 
@@ skipping 54 matching lines @@
   // The app window will show after 3 seconds.
   ExtensionTestMessageListener shown_listener("Shown", false);
   launched_listener.Reply("");
   EXPECT_TRUE(shown_listener.WaitUntilSatisfied());
   EXPECT_FALSE(app_window->is_hidden());
   EXPECT_TRUE(chrome::WillKeepAlive());
   app_window->GetBaseWindow()->Close();
 }
 
 #endif