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

Unified Diff: ash/mus/accelerators/accelerator_controller_delegate_mus.cc

Issue 2415163002: Expand and split DisplayController mojom. (Closed)
Patch Set: Created 4 years, 2 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
Index: ash/mus/accelerators/accelerator_controller_delegate_mus.cc
diff --git a/ash/mus/accelerators/accelerator_controller_delegate_mus.cc b/ash/mus/accelerators/accelerator_controller_delegate_mus.cc
index 803916cbcb99e527d0ffabac7833591a446f94b2..91aab634e2f860ee90b9f20fe0d6832ffa28b807 100644
--- a/ash/mus/accelerators/accelerator_controller_delegate_mus.cc
+++ b/ash/mus/accelerators/accelerator_controller_delegate_mus.cc
@@ -51,30 +51,20 @@ bool AcceleratorControllerDelegateMus::HandlesAction(AcceleratorAction action) {
return false;
#if defined(OS_CHROMEOS)
- case DEV_ADD_REMOVE_DISPLAY: {
- display::mojom::DisplayControllerPtr display_controller;
- connector_->ConnectToInterface("service:ui", &display_controller);
- display_controller->ToggleVirtualDisplay();
+ case DEV_ADD_REMOVE_DISPLAY:
+ case SWAP_PRIMARY_DISPLAY:
+ case TOUCH_HUD_PROJECTION_TOGGLE:
return true;
- }
case DEV_TOGGLE_UNIFIED_DESKTOP:
case LOCK_PRESSED:
case LOCK_RELEASED:
case POWER_PRESSED:
case POWER_RELEASED:
- case SWAP_PRIMARY_DISPLAY:
case TOGGLE_MIRROR_MODE:
case TOUCH_HUD_CLEAR:
case TOUCH_HUD_MODE_CHANGE:
NOTIMPLEMENTED();
return false;
- case TOUCH_HUD_PROJECTION_TOGGLE: {
- mash::mojom::LaunchablePtr launchable;
- connector_->ConnectToInterface("service:touch_hud", &launchable);
- launchable->Launch(mash::mojom::kWindow,
- mash::mojom::LaunchMode::DEFAULT);
- return true;
- }
#endif
default:
@@ -87,14 +77,47 @@ bool AcceleratorControllerDelegateMus::CanPerformAction(
AcceleratorAction action,
const ui::Accelerator& accelerator,
const ui::Accelerator& previous_accelerator) {
+ switch (action) {
+#if defined(OS_CHROMEOS)
+ case DEV_ADD_REMOVE_DISPLAY:
+ case SWAP_PRIMARY_DISPLAY:
+ case TOUCH_HUD_PROJECTION_TOGGLE:
+ return true;
+#endif
+ default:
+ break;
+ }
return false;
}
void AcceleratorControllerDelegateMus::PerformAction(
AcceleratorAction action,
const ui::Accelerator& accelerator) {
- // Should never be hit as HandlesAction() unconditionally returns false.
- NOTREACHED();
+ switch (action) {
+#if defined(OS_CHROMEOS)
+ case DEV_ADD_REMOVE_DISPLAY: {
+ display::mojom::DisplayControllerPtr display_controller;
+ connector_->ConnectToInterface("service:ui", &display_controller);
+ display_controller->ToggleAddRemoveDisplay();
+ break;
+ }
+ case SWAP_PRIMARY_DISPLAY: {
+ display::mojom::DisplayControllerPtr display_controller;
+ connector_->ConnectToInterface("service:ui", &display_controller);
+ display_controller->SwapPrimaryDisplay();
+ break;
+ }
+ case TOUCH_HUD_PROJECTION_TOGGLE: {
+ mash::mojom::LaunchablePtr launchable;
+ connector_->ConnectToInterface("service:touch_hud", &launchable);
+ launchable->Launch(mash::mojom::kWindow,
+ mash::mojom::LaunchMode::DEFAULT);
+ break;
+ }
+#endif
+ default:
+ NOTREACHED();
+ }
}
void AcceleratorControllerDelegateMus::ShowDeprecatedAcceleratorNotification(

Powered by Google App Engine
This is Rietveld 408576698