Commit dcbd9e3f authored by Sunggook Chue's avatar Sunggook Chue Committed by Chromium LUCI CQ

Enable ShortcutsMenu during Shortcut creation.

It looks like OsHookType::kShortcutsMenu set was omitted
in the recent refactoring of the code.

OsHookType::kShortcutsMenu should be set when kShortcuts
is set.

Bug: 1166362
Change-Id: I04020eef8e6deb804f0dff1903b2e994f179d8ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2628074Reviewed-by: default avatarDaniel Murphy <dmurph@chromium.org>
Reviewed-by: default avatarPhillis Tang <phillis@chromium.org>
Commit-Queue: Sunggook Chue <sunggch@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#843740}
parent c76f16f8
......@@ -823,6 +823,7 @@ void WebAppInstallTask::OnInstallFinalizedCreateShortcuts(
InstallOsHooksOptions options;
options.os_hooks[OsHookType::kShortcuts] = true;
options.os_hooks[OsHookType::kShortcutsMenu] = true;
options.add_to_desktop = true;
options.add_to_quick_launch_bar = kAddAppsToQuickLaunchBarByDefault;
options.os_hooks[OsHookType::kRunOnOsLogin] = web_app_info->run_on_os_login;
......@@ -839,12 +840,12 @@ void WebAppInstallTask::OnInstallFinalizedCreateShortcuts(
DCHECK(install_params_->locally_installed);
options.os_hooks[OsHookType::kShortcuts] =
install_params_->add_to_applications_menu;
options.add_to_desktop = install_params_->add_to_desktop;
options.add_to_quick_launch_bar = install_params_->add_to_quick_launch_bar;
options.os_hooks[OsHookType::kRunOnOsLogin] =
install_params_->run_on_os_login;
options.os_hooks[OsHookType::kShortcutsMenu] =
install_params_->add_to_applications_menu;
options.os_hooks[OsHookType::kRunOnOsLogin] =
install_params_->run_on_os_login;
options.add_to_desktop = install_params_->add_to_desktop;
options.add_to_quick_launch_bar = install_params_->add_to_quick_launch_bar;
}
auto hooks_created_callback =
......
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