Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2761)

Unified Diff: ash/accelerators/accelerator_controller_delegate_aura.cc

Issue 2282853002: ash: Add comment about where to add new accelerators (Closed)
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ash/accelerators/accelerator_controller_delegate_aura.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/accelerators/accelerator_controller_delegate_aura.cc
diff --git a/ash/accelerators/accelerator_controller_delegate_aura.cc b/ash/accelerators/accelerator_controller_delegate_aura.cc
index 654edd57b44a4a8ebadda05b631ca00f2ec11d44..efc09f6610e5eef2fee77d3bfe6678f9574515a1 100644
--- a/ash/accelerators/accelerator_controller_delegate_aura.cc
+++ b/ash/accelerators/accelerator_controller_delegate_aura.cc
@@ -361,6 +361,8 @@ void AcceleratorControllerDelegateAura::SetScreenshotDelegate(
bool AcceleratorControllerDelegateAura::HandlesAction(
AcceleratorAction action) {
+ // NOTE: When adding a new accelerator that only depends on //ash/common code,
+ // add it to accelerator_controller.cc instead. See class comment.
switch (action) {
case DEBUG_TOGGLE_DESKTOP_BACKGROUND_MODE:
case DEBUG_TOGGLE_DEVICE_SCALE_FACTOR:
« no previous file with comments | « ash/accelerators/accelerator_controller_delegate_aura.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698