Remove HostDesktopType from AppLaunchParams

chrome/browser/ui/extensions/app_launch_params.cc

 // Copyright 2014 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/ui/extensions/app_launch_params.h"
 
 #include "chrome/browser/extensions/launch_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "extensions/browser/extension_prefs.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
 #include "ui/base/window_open_disposition.h"
 
 using extensions::ExtensionPrefs;
 
 AppLaunchParams::AppLaunchParams(Profile* profile,
                                  const extensions::Extension* extension,
                                  extensions::LaunchContainer container,
                                  WindowOpenDisposition disposition,
                                  extensions::AppLaunchSource source)
     : 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),
-      source(source) {
+      source(source) {}
+
+AppLaunchParams CreateAppLaunchParamsUserContainer(
+    Profile* profile,
+    const extensions::Extension* extension,
+    WindowOpenDisposition disposition,
+    extensions::AppLaunchSource source) {
+  // Look up the app preference to find out the right launch container. Default
+  // is to launch as a regular tab.
+  extensions::LaunchContainer container =
+      extensions::GetLaunchContainer(ExtensionPrefs::Get(profile), extension);
+  return AppLaunchParams(profile, extension, container, disposition, source);
 }
 
-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
-  // 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),
-      source(source) {
+AppLaunchParams CreateAppLaunchParamsOverriddenContainer(

[benwells, 2016/03/16 06:29:43]

All but one of the callers of this call it with raw_disposition == CURRENT_TAB,
which seems equivalent to the above.

The one exception uses ui::DispositionFromEventFlags(event_flags). Instead of
having two constructors with the same parameter types, could this one take the
event_flags, look up the disposition from that, and then override the container
appropriately? That is, CreateAppLaunchParamsWithEventFlags or something.

[scottmg, 2016/03/16 16:38:54]

OK, for these 3, I switched to CreateAppLaunchParamsUseContainer() and switched
CURRENT_TAB to NEW_FOREGROUND_TAB, which is what the other function would have
done (I think).
Renamed the second factory as suggested.

+    Profile* profile,
+    const extensions::Extension* extension,
+    WindowOpenDisposition raw_disposition,
+    extensions::AppLaunchSource source) {
+  extensions::LaunchContainer container;
+  WindowOpenDisposition disposition;
   if (raw_disposition == NEW_FOREGROUND_TAB ||
       raw_disposition == NEW_BACKGROUND_TAB) {
     container = extensions::LAUNCH_CONTAINER_TAB;
     disposition = raw_disposition;
   } else if (raw_disposition == NEW_WINDOW) {
     container = extensions::LAUNCH_CONTAINER_WINDOW;
     disposition = raw_disposition;
   } else {
     // Look at preference to find the right launch container.  If no preference
     // is set, launch as a regular tab.
     container =
         extensions::GetLaunchContainer(ExtensionPrefs::Get(profile), extension);
     disposition = NEW_FOREGROUND_TAB;
   }
+  return AppLaunchParams(profile, extension, container, disposition, source);
 }
-
-AppLaunchParams::AppLaunchParams(const AppLaunchParams& other) = default;
-
-AppLaunchParams::~AppLaunchParams() {
-}