Expand and split DisplayController mojom.

ash/mus/accelerators/accelerator_controller_delegate_mus.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/mus/accelerators/accelerator_controller_delegate_mus.h"
 
 #include "base/logging.h"
 #include "mash/public/interfaces/launchable.mojom.h"
 #include "services/ui/public/interfaces/display/display_controller.mojom.h"
 
@@ skipping 33 matching lines @@
     case SCALE_UI_UP:
     case SHOW_SYSTEM_TRAY_BUBBLE:
     case TAKE_PARTIAL_SCREENSHOT:
     case TAKE_SCREENSHOT:
     case TAKE_WINDOW_SCREENSHOT:
     case UNPIN:
       NOTIMPLEMENTED();
       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:
       break;
   }
   return false;
 }
 
 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(
     const char* const notification_id,
     int message_id,
     int old_shortcut_id,
     int new_shortcut_id) {
   // TODO: http://crbug.com/630316.
   NOTIMPLEMENTED();
 }
 
 }  // namespace mus
 }  // namespace ash