For the Fullscreen magnifier, we have the two shortcuts Ctrl+Alt+BrightnessDown/BrightnessUp to decrease/increase the scale value. It is required that we make those same shortcuts work with the Docked Magnifier when it's enabled.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ba41da593b76dd6803f2d49256b872a2067e66d9 commit ba41da593b76dd6803f2d49256b872a2067e66d9 Author: Ahmed Fakhry <afakhry@chromium.org> Date: Thu Mar 01 18:10:40 2018 Docked Magnifier: Hook up zoom in/out shortcuts This CL makes the existing shortcuts affect whatever magnifier is currently enabled. This was also an opportunity to unify some scale related code. BUG= 817490 TEST=Added test coverage Change-Id: I366ee56629d09ae22c725b034bfc410077fe52ab Reviewed-on: https://chromium-review.googlesource.com/942052 Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Reviewed-by: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#540208} [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/BUILD.gn [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/accelerators/accelerator_controller.cc [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/accelerators/accelerator_table.cc [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/accelerators/accelerator_table.h [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/docked_magnifier_controller.cc [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/docked_magnifier_controller.h [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/magnification_controller.cc [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/magnification_controller.h [modify] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/magnification_controller_unittest.cc [add] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/magnifier_scale_utils.cc [add] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/magnifier_scale_utils.h [add] https://crrev.com/ba41da593b76dd6803f2d49256b872a2067e66d9/ash/magnifier/magnifier_scale_utils_unittest.cc
Comment 1 by afakhry@chromium.org
, Mar 1 2018