Accelerators: Make IsActionForAcceleratorEnabled() private
- It's only called by tests via ShellTestApi
- Make it private and call it via the existing TestApi
Bug: 1212168
Test: ash_unittests
Change-Id: I433489ebcaf456564bc76fc1f079b9e03e25dd1e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2987656
Commit-Queue: Zentaro Kavanagh <[email protected]>
Reviewed-by: James Cook <[email protected]>
Cr-Commit-Position: refs/heads/master@{#905903}
diff --git a/ash/shell_test_api.cc b/ash/shell_test_api.cc
index 90c910f..11886ec 100644
--- a/ash/shell_test_api.cc
+++ b/ash/shell_test_api.cc
@@ -276,8 +276,9 @@
bool ShellTestApi::IsActionForAcceleratorEnabled(
const ui::Accelerator& accelerator) const {
- return Shell::Get()->accelerator_controller()->IsActionForAcceleratorEnabled(
- accelerator);
+ auto* controller = Shell::Get()->accelerator_controller();
+ return AcceleratorControllerImpl::TestApi(controller)
+ .IsActionForAcceleratorEnabled(accelerator);
}
bool ShellTestApi::PressAccelerator(const ui::Accelerator& accelerator) {