Commit dfc6142e authored by Devlin Cronin's avatar Devlin Cronin Committed by Commit Bot

[UI Cleanup] Remove GetPrimaryChromeAcceleratorForBookmarkTab()

It's no longer used.

Bug: 894447
Change-Id: Ic81eb4d6bd34ad067e1f753af9300db444dc543b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129283
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: default avatarElly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#755065}
parent c9137cf9
......@@ -15,8 +15,6 @@ namespace chrome {
// Chrome.
bool IsChromeAccelerator(const ui::Accelerator& accelerator);
ui::Accelerator GetPrimaryChromeAcceleratorForBookmarkTab();
} // namespace chrome
#endif // CHROME_BROWSER_UI_ACCELERATOR_UTILS_H_
......@@ -6,7 +6,6 @@
#import <Cocoa/Cocoa.h>
#include "chrome/app/chrome_command_ids.h"
#include "chrome/browser/global_keyboard_shortcuts_mac.h"
#include "chrome/browser/ui/cocoa/accelerators_cocoa.h"
#include "ui/base/accelerators/accelerator.h"
......@@ -52,12 +51,4 @@ bool IsChromeAccelerator(const ui::Accelerator& accelerator) {
return CommandForKeyEvent(event).found();
}
ui::Accelerator GetPrimaryChromeAcceleratorForBookmarkTab() {
const ui::Accelerator* accelerator =
AcceleratorsCocoa::GetInstance()->GetAcceleratorForCommand(
IDC_BOOKMARK_THIS_TAB);
return accelerator ? *accelerator : ui::Accelerator();
}
} // namespace chrome
......@@ -4,7 +4,6 @@
#include <stddef.h>
#include "chrome/app/chrome_command_ids.h"
#include "chrome/browser/ui/views/accelerator_table.h"
#include "ui/base/accelerators/accelerator.h"
......@@ -35,16 +34,4 @@ bool IsChromeAccelerator(const ui::Accelerator& accelerator) {
return false;
}
ui::Accelerator GetPrimaryChromeAcceleratorForBookmarkTab() {
std::vector<AcceleratorMapping> accelerators = GetAcceleratorList();
for (size_t i = 0; i < accelerators.size(); ++i) {
if (accelerators[i].command_id == IDC_BOOKMARK_THIS_TAB) {
return ui::Accelerator(accelerators[i].keycode,
accelerators[i].modifiers);
}
}
return ui::Accelerator();
}
} // namespace chrome
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