Commit 0a0fc3d4 authored by Daniel Murphy's avatar Daniel Murphy Committed by Commit Bot

[BMO] Chrome://apps uninstall dialog now tied to browser window

I accidentally didn't associate the uninstall window to the browser
window when updating the uninstall code to support BMO.

R=calamity@chromium.org, loyso@chromium.org

Fixed: 1070159
Change-Id: I2fa4e50bd28af7283b57491d0e49c3a60350566a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2147545
Auto-Submit: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: calamity <calamity@chromium.org>
Reviewed-by: default avatarAlexey Baskakov <loyso@chromium.org>
Reviewed-by: default avatarcalamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#758742}
parent 721fda2b
...@@ -876,13 +876,14 @@ void AppLauncherHandler::HandleUninstallApp(const base::ListValue* args) { ...@@ -876,13 +876,14 @@ void AppLauncherHandler::HandleUninstallApp(const base::ListValue* args) {
web_app_provider_->install_finalizer().UninstallExternalAppByUser( web_app_provider_->install_finalizer().UninstallExternalAppByUser(
extension_id_prompting_, std::move(uninstall_success_callback)); extension_id_prompting_, std::move(uninstall_success_callback));
} else { } else {
Browser* browser =
chrome::FindBrowserWithWebContents(web_ui()->GetWebContents());
web_app::WebAppUiManagerImpl::Get(Profile::FromWebUI(web_ui())) web_app::WebAppUiManagerImpl::Get(Profile::FromWebUI(web_ui()))
->dialog_manager() ->dialog_manager()
.UninstallWebApp( .UninstallWebApp(
extension_id_prompting_, extension_id_prompting_,
web_app::WebAppDialogManager::UninstallSource::kAppsPage, web_app::WebAppDialogManager::UninstallSource::kAppsPage,
/*browser_window=*/nullptr, browser->window(), std::move(uninstall_success_callback));
std::move(uninstall_success_callback));
} }
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment