Commit 02f738b6 authored by Mike Jackson's avatar Mike Jackson Committed by Chromium LUCI CQ

dpwas: Clean up TODO comment for Run on OS Login UX

Per UX review, the only planned place to expose Run on OS Login
is via the context menu option on a web app listed in chrome://apps
list. This change cleans up a code comment indicating it might be
surfaced elsewhere in the UI.

Bug: 1110407
Change-Id: I43c5987d18ba42081b88eea723a776d556ea839c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633878Reviewed-by: default avatarChase Phillips <cmp@chromium.org>
Commit-Queue: Mike Jackson <mjackson@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#846254}
parent a7fe60e4
...@@ -83,9 +83,6 @@ void WebAppMenuModel::ExecuteCommand(int command_id, int event_flags) { ...@@ -83,9 +83,6 @@ void WebAppMenuModel::ExecuteCommand(int command_id, int event_flags) {
} }
void WebAppMenuModel::Build() { void WebAppMenuModel::Build() {
// TODO(crbug.com/897302): Expose UI for user opt out and reenable for the Run
// on OS Login feature.
if (CreateActionToolbarOverflowMenu()) if (CreateActionToolbarOverflowMenu())
AddSeparator(ui::UPPER_SEPARATOR); AddSeparator(ui::UPPER_SEPARATOR);
AddItemWithStringId(IDC_WEB_APP_MENU_APP_INFO, AddItemWithStringId(IDC_WEB_APP_MENU_APP_INFO,
......
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