| commit | 0301854e45c50b194b590de17a3f6801fb4ae9cc | [log] [tgz] |
|---|---|---|
| author | Johan Bay <jobay@chromium.org> | Thu Jun 10 22:05:46 2021 |
| committer | Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> | Thu Jun 10 22:05:46 2021 |
| tree | 3771451d45044e9ccad49b5fc35a8ddab31617d8 | |
| parent | 151b1922a11c8391a0068574b4c4e26026154aff [diff] |
Remove "Inspect" context menu shortcut label The current shortcut label is wrong: The shortcut Ctrl+Shift+I does not inspect an element, but instead toggles DevTools. There currently is no keyboard accessible way of performing the equivalent action. This change removes shortcut label with the intent to bring it back when an equivalent keyboard shortcut exists. Doc: https://docs.google.com/document/d/1NcGC1TYLqci1N7fCvyf1BPI8BzMxU2VTQN4aR8V5dU4/ Bug: 1203618 Change-Id: Ib0c2868ee3cd356eb23f5db5fe0e3a105ba5012a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2949330 Commit-Queue: Dana Fried <dfried@chromium.org> Reviewed-by: Dana Fried <dfried@chromium.org> Reviewed-by: Mathias Bynens <mathias@chromium.org> Cr-Commit-Position: refs/heads/master@{#891405}
diff --git a/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc b/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc index 0271d19..13a65de 100644 --- a/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc +++ b/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc
@@ -161,11 +161,6 @@ *accel = ui::Accelerator(ui::VKEY_C, ui::EF_CONTROL_DOWN); return true; - case IDC_CONTENT_CONTEXT_INSPECTELEMENT: - *accel = ui::Accelerator(ui::VKEY_I, - ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN); - return true; - case IDC_CONTENT_CONTEXT_PASTE: *accel = ui::Accelerator(ui::VKEY_V, ui::EF_CONTROL_DOWN); return true;