diff options
author | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2021-10-26 13:57:00 +0200 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2021-11-02 11:31:01 +0000 |
commit | 1943b3c2a1dcee36c233724fc4ee7613d71b9cf6 (patch) | |
tree | 8c1b5f12357025c197da5427ae02cfdc2f3570d6 /chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc | |
parent | 21ba0c5d4bf8fba15dddd97cd693bad2358b77fd (diff) |
BASELINE: Update Chromium to 94.0.4606.111
Change-Id: I924781584def20fc800bedf6ff41fdb96c438193
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc')
-rw-r--r-- | chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc | 35 |
1 files changed, 20 insertions, 15 deletions
diff --git a/chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc b/chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc index c8479274c47..b172325913a 100644 --- a/chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc +++ b/chromium/chrome/browser/extensions/api/management/chrome_management_api_delegate.cc @@ -32,14 +32,15 @@ #include "chrome/browser/ui/tab_helpers.h" #include "chrome/browser/ui/web_applications/web_app_dialog_utils.h" #include "chrome/browser/ui/webui/extensions/extension_icon_source.h" -#include "chrome/browser/web_applications/components/app_registrar.h" -#include "chrome/browser/web_applications/components/install_manager.h" #include "chrome/browser/web_applications/components/web_app_constants.h" #include "chrome/browser/web_applications/components/web_app_helpers.h" #include "chrome/browser/web_applications/components/web_app_install_utils.h" -#include "chrome/browser/web_applications/components/web_app_provider_base.h" #include "chrome/browser/web_applications/components/web_app_utils.h" #include "chrome/browser/web_applications/components/web_application_info.h" +#include "chrome/browser/web_applications/web_app_install_manager.h" +#include "chrome/browser/web_applications/web_app_install_params.h" +#include "chrome/browser/web_applications/web_app_provider.h" +#include "chrome/browser/web_applications/web_app_registrar.h" #include "chrome/common/extensions/extension_metrics.h" #include "chrome/common/extensions/manifest_handlers/app_launch_info.h" #include "components/favicon/core/favicon_service.h" @@ -91,7 +92,7 @@ using InstallOrLaunchWebAppCallback = extensions::ManagementAPIDelegate::InstallOrLaunchWebAppCallback; using InstallOrLaunchWebAppResult = extensions::ManagementAPIDelegate::InstallOrLaunchWebAppResult; -using InstallableCheckResult = web_app::InstallManager::InstallableCheckResult; +using InstallableCheckResult = web_app::InstallableCheckResult; #if BUILDFLAG(IS_CHROMEOS_ASH) void OnDidCheckForIntentToPlayStore(const std::string& intent, @@ -144,8 +145,9 @@ class ManagementSetEnabledFunctionInstallPromptDelegate ~ManagementSetEnabledFunctionInstallPromptDelegate() override {} private: - void OnInstallPromptDone(ExtensionInstallPrompt::Result result) { - std::move(callback_).Run(result == + void OnInstallPromptDone( + ExtensionInstallPrompt::DoneCallbackPayload payload) { + std::move(callback_).Run(payload.result == ExtensionInstallPrompt::Result::ACCEPTED); } @@ -247,8 +249,8 @@ class ChromeAppForLinkDelegate : public extensions::AppForLinkDelegate { image_result.image.AsBitmap(); } - auto* provider = web_app::WebAppProviderBase::GetProviderBase( - Profile::FromBrowserContext(context)); + auto* provider = + web_app::WebAppProvider::Get(Profile::FromBrowserContext(context)); DCHECK(provider); provider->install_manager().InstallWebAppFromInfo( @@ -261,10 +263,10 @@ class ChromeAppForLinkDelegate : public extensions::AppForLinkDelegate { extensions::api::management::ExtensionInfo CreateExtensionInfoFromWebApp( const std::string& app_id, content::BrowserContext* context) override { - auto* provider = web_app::WebAppProviderBase::GetProviderBase( - Profile::FromBrowserContext(context)); + auto* provider = + web_app::WebAppProvider::Get(Profile::FromBrowserContext(context)); DCHECK(provider); - const web_app::AppRegistrar& registrar = provider->registrar(); + const web_app::WebAppRegistrar& registrar = provider->registrar(); extensions::api::management::ExtensionInfo info; info.id = app_id; @@ -304,8 +306,9 @@ class ChromeAppForLinkDelegate : public extensions::AppForLinkDelegate { info.launch_type = extensions::api::management::LAUNCH_TYPE_OPEN_FULL_SCREEN; break; - // This mode is not supported by the extension app backend. + // These modes are not supported by the extension app backend. case web_app::DisplayMode::kWindowControlsOverlay: + case web_app::DisplayMode::kTabbed: case web_app::DisplayMode::kUndefined: info.launch_type = extensions::api::management::LAUNCH_TYPE_NONE; break; @@ -326,7 +329,7 @@ void LaunchWebApp(const web_app::AppId& app_id, Profile* profile) { // preference, the default launch value will be returned. // TODO(crbug.com/1003602): Make AppLaunchParams launch container Optional or // add a "default" launch container enum value. - auto* provider = web_app::WebAppProviderBase::GetProviderBase(profile); + auto* provider = web_app::WebAppProvider::Get(profile); DCHECK(provider); blink::mojom::DisplayMode display_mode = provider->registrar().GetAppUserDisplayMode(app_id); @@ -529,13 +532,15 @@ void ChromeManagementAPIDelegate::InstallOrLaunchReplacementWebApp( const GURL& web_app_url, InstallOrLaunchWebAppCallback callback) const { Profile* profile = Profile::FromBrowserContext(context); - auto* provider = web_app::WebAppProviderBase::GetProviderBase(profile); + auto* provider = web_app::WebAppProvider::Get(profile); DCHECK(provider); // Launch the app if web_app_url happens to match start_url. If not, the app // could still be installed with different start_url. if (provider->registrar().IsLocallyInstalled(web_app_url)) { - LaunchWebApp(web_app::GenerateAppIdFromURL(web_app_url), profile); + LaunchWebApp( + web_app::GenerateAppId(/*manifest_id=*/absl::nullopt, web_app_url), + profile); std::move(callback).Run(InstallOrLaunchWebAppResult::kSuccess); return; } |