Remove HostDesktopType from AppLaunchParams

chrome/browser/apps/app_browsertest_util.cc

Index: chrome/browser/apps/app_browsertest_util.cc
diff --git a/chrome/browser/apps/app_browsertest_util.cc b/chrome/browser/apps/app_browsertest_util.cc
index bb196148f2c2d7dd3863e49f3dfcf04278be71cf..64d8b49c6ebc470870519fe9517bb27bbc51b82a 100644
--- a/chrome/browser/apps/app_browsertest_util.cc
+++ b/chrome/browser/apps/app_browsertest_util.cc
@@ -121,9 +121,9 @@ const Extension* PlatformAppBrowserTest::InstallAndLaunchPlatformApp(
 }
 
 void PlatformAppBrowserTest::LaunchPlatformApp(const Extension* extension) {
-  OpenApplication(AppLaunchParams(browser()->profile(), extension,
-                                  LAUNCH_CONTAINER_NONE, NEW_WINDOW,
-                                  extensions::SOURCE_TEST));
+  OpenApplication(CreateAppLaunchParamsUserContainer(
+      browser()->profile(), extension, LAUNCH_CONTAINER_NONE, NEW_WINDOW,

[benwells, 2016/03/16 22:50:12]

Does this compile? I think it should stay as it wsa, as the container is passed
in.

[scottmg, 2016/03/17 00:24:11]

Thanks. I used cs.chromium.org to (attempt to) xref to the correct overrides of
the AppLaunchParams ctors, but when I popped over to my editor I changed the
wrong one (LaunchPlatformApp() instead of LaunchHostedApp()). Reverted here and
fixed below.

+      extensions::SOURCE_TEST));
 }
 
 void PlatformAppBrowserTest::LaunchHostedApp(const Extension* extension) {