desktop-pwas: Add SetAppLaunchContainer method to AppRegistrar.

Only implemented for extensions for now.
Will be used in subsequent CL.

Bug: 989792
Change-Id: I9d15f7c007ecdf655c060d858122a7b3b3ed768a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783679
Commit-Queue: Glen Robertson <glenrob@chromium.org>
Auto-Submit: Glen Robertson <glenrob@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#692987}

chrome/browser/web_applications/components/app_registrar.h

@@ -90,6 +90,8 @@ class AppRegistrar {
   virtual const GURL& GetAppLaunchURL(const AppId& app_id) const = 0;
   virtual base::Optional<GURL> GetAppScope(const AppId& app_id) const = 0;
   virtual LaunchContainer GetAppLaunchContainer(const AppId& app_id) const = 0;
+  virtual void SetAppLaunchContainer(const AppId& app_id,
+                                     LaunchContainer launch_container) = 0;
 
   virtual std::vector<AppId> GetAppIds() const = 0;
 

chrome/browser/web_applications/extensions/bookmark_app_registrar.cc

@@ -164,6 +164,29 @@ web_app::LaunchContainer BookmarkAppRegistrar::GetAppLaunchContainer(
   }
 }
 
+void BookmarkAppRegistrar::SetAppLaunchContainer(
+    const web_app::AppId& app_id,
+    web_app::LaunchContainer launch_container) {
+  const Extension* extension = GetExtension(app_id);
+  DCHECK(extension);
+  if (!extension)
+    return;
+
+  switch (launch_container) {
+    case web_app::LaunchContainer::kWindow:
+      extensions::SetLaunchType(profile(), extension->id(),
+                                extensions::LAUNCH_TYPE_WINDOW);
+      return;
+    case web_app::LaunchContainer::kTab:
+      extensions::SetLaunchType(profile(), extension->id(),
+                                extensions::LAUNCH_TYPE_REGULAR);
+      return;
+    case web_app::LaunchContainer::kDefault:
+      NOTREACHED();
+      return;
+  }
+}
+
 std::vector<web_app::AppId> BookmarkAppRegistrar::GetAppIds() const {
   std::vector<web_app::AppId> app_ids;
   for (scoped_refptr<const Extension> app :

chrome/browser/web_applications/extensions/bookmark_app_registrar.h

@@ -42,6 +42,9 @@ class BookmarkAppRegistrar : public web_app::AppRegistrar,
   base::Optional<GURL> GetAppScope(const web_app::AppId& app_id) const override;
   web_app::LaunchContainer GetAppLaunchContainer(
       const web_app::AppId& app_id) const override;
+  void SetAppLaunchContainer(
+      const web_app::AppId& app_id,
+      web_app::LaunchContainer launch_container) override;
   std::vector<web_app::AppId> GetAppIds() const override;
 
   // ExtensionRegistryObserver:

chrome/browser/web_applications/test/test_app_registrar.cc

@@ -137,6 +137,11 @@ LaunchContainer TestAppRegistrar::GetAppLaunchContainer(
   return LaunchContainer::kTab;
 }
 
+void TestAppRegistrar::SetAppLaunchContainer(const AppId& app_id,
+                                             LaunchContainer launch_container) {
+  NOTIMPLEMENTED();
+}
+
 std::vector<AppId> TestAppRegistrar::GetAppIds() const {
   std::vector<AppId> result;
   for (const std::pair<AppId, AppInfo>& it : installed_apps_) {

chrome/browser/web_applications/test/test_app_registrar.h

@@ -60,6 +60,8 @@ class TestAppRegistrar : public AppRegistrar {
   base::Optional<GURL> GetAppScope(const AppId& app_id) const override;
   web_app::LaunchContainer GetAppLaunchContainer(
       const web_app::AppId& app_id) const override;
+  void SetAppLaunchContainer(const AppId& app_id,
+                             LaunchContainer launch_container) override;
   std::vector<AppId> GetAppIds() const override;
 
  private:

chrome/browser/web_applications/web_app_registrar.cc

@@ -146,6 +146,11 @@ LaunchContainer WebAppRegistrar::GetAppLaunchContainer(
   return web_app ? web_app->launch_container() : LaunchContainer::kDefault;
 }
 
+void WebAppRegistrar::SetAppLaunchContainer(const AppId& app_id,
+                                            LaunchContainer launch_container) {
+  NOTIMPLEMENTED();
+}
+
 std::vector<AppId> WebAppRegistrar::GetAppIds() const {
   std::vector<AppId> app_ids;
   app_ids.reserve(registry_.size());

chrome/browser/web_applications/web_app_registrar.h

@@ -50,6 +50,8 @@ class WebAppRegistrar : public AppRegistrar {
   const GURL& GetAppLaunchURL(const AppId& app_id) const override;
   base::Optional<GURL> GetAppScope(const AppId& app_id) const override;
   LaunchContainer GetAppLaunchContainer(const AppId& app_id) const override;
+  void SetAppLaunchContainer(const AppId& app_id,
+                             LaunchContainer launch_container) override;
   std::vector<AppId> GetAppIds() const override;
 
   // Only range-based |for| loop supported. Don't use AppSet directly.