Add support for deferring app initialization when testing.

chrome/test/remoting/remote_desktop_browsertest.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/test/remoting/remote_desktop_browsertest.h"
 
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/json/json_reader.h"
 #include "base/path_service.h"
@@ skipping 148 matching lines @@
     extensions::Manifest::Type type = extension_->GetType();
     EXPECT_TRUE(type == extensions::Manifest::TYPE_PLATFORM_APP ||
                 type == extensions::Manifest::TYPE_LEGACY_PACKAGED_APP);
 
     EXPECT_TRUE(extension_->ShouldDisplayInAppLauncher());
   }
 
   ASSERT_EQ(installed, expected);
 }
 
-void RemoteDesktopBrowserTest::LaunchChromotingApp() {
+void RemoteDesktopBrowserTest::LaunchChromotingApp(bool defer_start) {
   ASSERT_TRUE(extension_);
 
   GURL chromoting_main = Chromoting_Main_URL();
   // We cannot simply wait for any page load because the first page
   // loaded could be the generated background page. We need to wait
   // till the chromoting main page is loaded.
   PageLoadNotificationObserver observer(chromoting_main);
+  observer.set_ignore_url_parameters(true);
+
+  // Start the app in deferred mode by setting a URL parameter.
+  scoped_ptr<extensions::FeatureSwitch::ScopedOverride> enable_trace_app_source;
+  if (defer_start) {
+    enable_trace_app_source.reset(
+        new extensions::FeatureSwitch::ScopedOverride(
+            extensions::FeatureSwitch::trace_app_source(),
+            true));
+  }

[weitao, 2014/12/12 22:49:52]

Where is the "test" URL parameter added?

[Jamie, 2014/12/13 01:03:43]

extensions::FeatureSwitch::trace_app_source() is a command-line option that
causes apps to be launched with a |source| URL parameter. The source is
specified in AppLaunchParams; specifically, in this case it's specified on line
193. If |defer_start| is true, we force that parameter to true.

[weitao, 2014/12/16 01:06:27]

I see. The combination of trace_app_source being true and SOURCE_TEST passed to
AppLaunchParams will add "source='test'" to the URL. It took me a moment to
understand that. Could you please add a short comment explaining that?

[Jamie, 2014/12/16 01:33:51]

Done.

 
   OpenApplication(AppLaunchParams(browser()->profile(), extension_,
                                   is_platform_app()
                                       ? extensions::LAUNCH_CONTAINER_NONE
                                       : extensions::LAUNCH_CONTAINER_TAB,
                                   is_platform_app() ? NEW_WINDOW : CURRENT_TAB,
                                   extensions::SOURCE_TEST));
 
   observer.Wait();
 
@@ skipping 16 matching lines @@
     // event. A half second wait is necessary for the subsequent javascript
     // injection to work.
     // TODO(weitaosu): Find out whether there is a more appropriate notification
     // to wait for so we can get rid of this wait.
     ASSERT_TRUE(TimeoutWaiter(base::TimeDelta::FromSeconds(5)).Wait());
   }
 
   EXPECT_EQ(Chromoting_Main_URL(), GetCurrentURL());
 }
 
+void RemoteDesktopBrowserTest::StartChromotingApp() {
+  ClickOnControl("browser-test-continue-init");
+};
+
 void RemoteDesktopBrowserTest::Authorize() {
   // The chromoting extension should be installed.
   ASSERT_TRUE(extension_);
 
   // The chromoting main page should be loaded in the current tab
   // and isAuthenticated() should be false (auth dialog visible).
   ASSERT_EQ(Chromoting_Main_URL(), GetCurrentURL());
   ASSERT_FALSE(IsAuthenticated());
 
   // The second observer monitors the loading of the Google login page.
@@ skipping 235 matching lines @@
   // TODO(chaitali): Remove this additional timeout after we figure out
   // why this is needed for the v1 app to work.
   // Without this timeout the test fail with a "CloseWebContents called for
   // tab not in our strip" error for the v1 app.
   ASSERT_TRUE(TimeoutWaiter(base::TimeDelta::FromSeconds(2)).Wait());
 }
 
 void RemoteDesktopBrowserTest::SetUpTestForMe2Me() {
   VerifyInternetAccess();
   Install();
-  LaunchChromotingApp();
+  LaunchChromotingApp(false);
   Auth();
   LoadScript(app_web_content(), FILE_PATH_LITERAL("browser_test.js"));
   ExpandMe2Me();
   // The call to EnsureRemoteConnectionEnabled() does a PIN reset.
   // This causes the test to fail because of a recent bug:
   // crbug.com/430676
   // TODO(anandc): Reactivate this call after above bug is fixed.
   //EnsureRemoteConnectionEnabled();
 }
 
@@ skipping 408 matching lines @@
 // static
 bool RemoteDesktopBrowserTest::IsEnabled(
     content::WebContents* client_web_content,
     const std::string& element_name) {
   return !ExecuteScriptAndExtractBool(
     client_web_content,
     "document.getElementById(\"" + element_name + "\").disabled");
 }
 
 }  // namespace remoting