Remove HostDesktopType from AppLaunchParams

chrome/browser/ui/extensions/app_launch_params.cc

Index: chrome/browser/ui/extensions/app_launch_params.cc
diff --git a/chrome/browser/ui/extensions/app_launch_params.cc b/chrome/browser/ui/extensions/app_launch_params.cc
index 80dab730223928003e4ac3a77f839fa98226c5bc..f099ad59b45acfa2f46e1c8b7148c33a86b21d96 100644
--- a/chrome/browser/ui/extensions/app_launch_params.cc
+++ b/chrome/browser/ui/extensions/app_launch_params.cc
@@ -22,7 +22,6 @@ AppLaunchParams::AppLaunchParams(Profile* profile,
       extension_id(extension ? extension->id() : std::string()),
       container(container),
       disposition(disposition),
-      desktop_type(chrome::GetActiveDesktop()),
       override_url(),
       override_bounds(),
       command_line(base::CommandLine::NO_PROGRAM),
@@ -31,32 +30,11 @@ AppLaunchParams::AppLaunchParams(Profile* profile,
 
 AppLaunchParams::AppLaunchParams(Profile* profile,
                                  const extensions::Extension* extension,
-                                 WindowOpenDisposition disposition,
-                                 extensions::AppLaunchSource source)
-    : profile(profile),
-      extension_id(extension ? extension->id() : std::string()),
-      container(extensions::LAUNCH_CONTAINER_NONE),
-      disposition(disposition),
-      desktop_type(chrome::GetActiveDesktop()),
-      override_url(),
-      override_bounds(),
-      command_line(base::CommandLine::NO_PROGRAM),
-      source(source) {
-  // Look up the app preference to find out the right launch container. Default

[scottmg, 2016/02/09 22:50:04]

This was a bit weird. The constructor that took a HostDesktopType did a bunch of
extra stuff, but that extra stuff is unrelated to |desktop_type|. I kept the one
that did more.

[scottmg, 2016/02/17 17:33:42]

It looks like this wasn't right. The two constructor's args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

-  // is to launch as a regular tab.
-  container =
-      extensions::GetLaunchContainer(ExtensionPrefs::Get(profile), extension);
-}
-
-AppLaunchParams::AppLaunchParams(Profile* profile,
-                                 const extensions::Extension* extension,
                                  WindowOpenDisposition raw_disposition,
-                                 chrome::HostDesktopType desktop_type,
                                  extensions::AppLaunchSource source)
     : profile(profile),
       extension_id(extension ? extension->id() : std::string()),
       container(extensions::LAUNCH_CONTAINER_NONE),
-      desktop_type(desktop_type),
       override_url(),
       override_bounds(),
       command_line(base::CommandLine::NO_PROGRAM),