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

Side by Side Diff: ash/mus/accelerators/accelerator_controller_delegate_mus.cc

Issue 2323863002: Separate debugging and developer accelerators (Closed)
Patch Set: Oshima's comments AND Rebase Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ash/mus/accelerators/accelerator_controller_delegate_mus.h" 5 #include "ash/mus/accelerators/accelerator_controller_delegate_mus.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "mash/public/interfaces/launchable.mojom.h" 8 #include "mash/public/interfaces/launchable.mojom.h"
9 #include "services/ui/public/interfaces/display/display_controller.mojom.h" 9 #include "services/ui/public/interfaces/display/display_controller.mojom.h"
10 10
(...skipping 13 matching lines...) Expand all
24 AcceleratorControllerDelegateMus::~AcceleratorControllerDelegateMus() {} 24 AcceleratorControllerDelegateMus::~AcceleratorControllerDelegateMus() {}
25 25
26 bool AcceleratorControllerDelegateMus::HandlesAction(AcceleratorAction action) { 26 bool AcceleratorControllerDelegateMus::HandlesAction(AcceleratorAction action) {
27 // This is the list of actions that are not ported from aura. The actions are 27 // This is the list of actions that are not ported from aura. The actions are
28 // replicated here to make sure we don't forget any. This list should 28 // replicated here to make sure we don't forget any. This list should
29 // eventually be empty. If there are any actions that don't make sense for 29 // eventually be empty. If there are any actions that don't make sense for
30 // mus, then they should be removed from AcceleratorAction. 30 // mus, then they should be removed from AcceleratorAction.
31 // http://crbug.com/612331. 31 // http://crbug.com/612331.
32 switch (action) { 32 switch (action) {
33 case DEBUG_TOGGLE_DEVICE_SCALE_FACTOR: 33 case DEBUG_TOGGLE_DEVICE_SCALE_FACTOR:
34 case DEBUG_TOGGLE_ROOT_WINDOW_FULL_SCREEN: 34 case DEV_TOGGLE_ROOT_WINDOW_FULL_SCREEN:
35 case DEBUG_TOGGLE_SHOW_DEBUG_BORDERS: 35 case DEBUG_TOGGLE_SHOW_DEBUG_BORDERS:
36 case DEBUG_TOGGLE_SHOW_FPS_COUNTER: 36 case DEBUG_TOGGLE_SHOW_FPS_COUNTER:
37 case DEBUG_TOGGLE_SHOW_PAINT_RECTS: 37 case DEBUG_TOGGLE_SHOW_PAINT_RECTS:
38 case MAGNIFY_SCREEN_ZOOM_IN: 38 case MAGNIFY_SCREEN_ZOOM_IN:
39 case MAGNIFY_SCREEN_ZOOM_OUT: 39 case MAGNIFY_SCREEN_ZOOM_OUT:
40 case ROTATE_SCREEN: 40 case ROTATE_SCREEN:
41 case ROTATE_WINDOW: 41 case ROTATE_WINDOW:
42 case SCALE_UI_DOWN: 42 case SCALE_UI_DOWN:
43 case SCALE_UI_RESET: 43 case SCALE_UI_RESET:
44 case SCALE_UI_UP: 44 case SCALE_UI_UP:
45 case SHOW_SYSTEM_TRAY_BUBBLE: 45 case SHOW_SYSTEM_TRAY_BUBBLE:
46 case TAKE_PARTIAL_SCREENSHOT: 46 case TAKE_PARTIAL_SCREENSHOT:
47 case TAKE_SCREENSHOT: 47 case TAKE_SCREENSHOT:
48 case TAKE_WINDOW_SCREENSHOT: 48 case TAKE_WINDOW_SCREENSHOT:
49 case UNPIN: 49 case UNPIN:
50 NOTIMPLEMENTED(); 50 NOTIMPLEMENTED();
51 return false; 51 return false;
52 52
53 #if defined(OS_CHROMEOS) 53 #if defined(OS_CHROMEOS)
54 case DEBUG_ADD_REMOVE_DISPLAY: { 54 case DEV_ADD_REMOVE_DISPLAY: {
55 display::mojom::DisplayControllerPtr display_controller; 55 display::mojom::DisplayControllerPtr display_controller;
56 connector_->ConnectToInterface("mojo:ui", &display_controller); 56 connector_->ConnectToInterface("mojo:ui", &display_controller);
57 display_controller->ToggleVirtualDisplay(); 57 display_controller->ToggleVirtualDisplay();
58 break; 58 break;
59 } 59 }
60 case DEBUG_TOGGLE_UNIFIED_DESKTOP: 60 case DEV_TOGGLE_UNIFIED_DESKTOP:
61 case DISABLE_GPU_WATCHDOG: 61 case DISABLE_GPU_WATCHDOG:
62 case LOCK_PRESSED: 62 case LOCK_PRESSED:
63 case LOCK_RELEASED: 63 case LOCK_RELEASED:
64 case POWER_PRESSED: 64 case POWER_PRESSED:
65 case POWER_RELEASED: 65 case POWER_RELEASED:
66 case SWAP_PRIMARY_DISPLAY: 66 case SWAP_PRIMARY_DISPLAY:
67 case TOGGLE_MIRROR_MODE: 67 case TOGGLE_MIRROR_MODE:
68 case TOUCH_HUD_CLEAR: 68 case TOUCH_HUD_CLEAR:
69 case TOUCH_HUD_MODE_CHANGE: 69 case TOUCH_HUD_MODE_CHANGE:
70 NOTIMPLEMENTED(); 70 NOTIMPLEMENTED();
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 const char* const notification_id, 102 const char* const notification_id,
103 int message_id, 103 int message_id,
104 int old_shortcut_id, 104 int old_shortcut_id,
105 int new_shortcut_id) { 105 int new_shortcut_id) {
106 // TODO: http://crbug.com/630316. 106 // TODO: http://crbug.com/630316.
107 NOTIMPLEMENTED(); 107 NOTIMPLEMENTED();
108 } 108 }
109 109
110 } // namespace mus 110 } // namespace mus
111 } // namespace ash 111 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698