New issue
Advanced search Search tips

Issue 827651 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task
M-X



Sign in to add a comment

Command::AcceleratorToString duplicates Accelerator::GetShortcutText()

Project Member Reported by ellyjo...@chromium.org, Mar 30 2018

Issue description

These two should be folded together.
 

Comment 1 by a...@chromium.org, Mar 30 2018

Cc: a...@chromium.org
Components: Platform>Extensions
The code's rationale is:

  // Returns the string representation of an accelerator without localizing the
  // shortcut text (like accelerator::GetShortcutText() does).

Nevertheless, less duplication is better.

Sign in to add a comment