Full-screen Magnifier: Support warping the cursor on the edge

ash/accelerators/accelerator_controller.cc

 // Copyright (c) 2012 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/accelerators/accelerator_controller.h"
 
 #include "ash/accelerators/accelerator_table.h"
 #include "ash/ash_switches.h"
 #include "ash/caps_lock_delegate.h"
 #include "ash/desktop_background/desktop_background_controller.h"
 #include "ash/focus_cycler.h"
 #include "ash/ime_control_delegate.h"
 #include "ash/launcher/launcher.h"
 #include "ash/launcher/launcher_delegate.h"
 #include "ash/launcher/launcher_model.h"
+#include "ash/magnifier/magnification_controller.h"
 #include "ash/monitor/multi_monitor_manager.h"
 #include "ash/screenshot_delegate.h"
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
 #include "ash/shell_window_ids.h"
 #include "ash/system/brightness/brightness_control_delegate.h"
 #include "ash/system/tray/system_tray.h"
 #include "ash/volume_control_delegate.h"
 #include "ash/wm/property_util.h"
 #include "ash/wm/window_cycle_controller.h"
@@ skipping 138 matching lines @@
 }
 
 bool HandlePrintWindowHierarchy() {
   VLOG(1) << "Window hierarchy:";
   aura::Window* container = ash::Shell::GetInstance()->GetContainer(
       ash::internal::kShellWindowId_DefaultContainer);
   PrintWindowHierarchy(container, 0);
   return true;
 }
 
+// Mafnify the screen
+bool HandleMagnifyScreen(int delta_index) {
+  float scale = (delta_index > 0) ? 1.2f : 1.0f/1.2f;

[sky, 2012/06/07 18:05:50]

Use constants (and spaces).

[yoshiki, 2012/06/08 22:27:45]

Done.

+
+  float curren_scale = ash::Shell::GetInstance()->

[sky, 2012/06/07 18:05:50]

current_scale

[yoshiki, 2012/06/08 22:27:45]

Done.

+      magnification_controller()->GetScale();
+  ash::Shell::GetInstance()->
+      magnification_controller()->SetScale(curren_scale * scale, true);

[sky, 2012/06/07 18:05:50]

Won't this lead to rounding errors? Wouldn't it be better to maintain scale as
an int and calculate the float value from that?

[yoshiki, 2012/06/08 22:27:45]

Done.

+
+  return true;
+}
+
 #endif
 
 }  // namespace
 
 namespace ash {
 
 ////////////////////////////////////////////////////////////////////////////////
 // AcceleratorController, public:
 
 AcceleratorController::AcceleratorController()
@@ skipping 279 matching lines @@
       return HandlePrintWindowHierarchy();
     case MONITOR_ADD_REMOVE:
       internal::MultiMonitorManager::AddRemoveMonitor();
       return true;
     case MONITOR_CYCLE:
       internal::MultiMonitorManager::CycleMonitor();
       return true;
     case MONITOR_TOGGLE_SCALE:
       internal::MultiMonitorManager::ToggleMonitorScale();
       return true;
+    case MAGNIFY_SCREEN_ZOOM_IN:
+      return HandleMagnifyScreen(1);
+    case MAGNIFY_SCREEN_ZOOM_OUT:
+      return HandleMagnifyScreen(-1);
 #endif
     default:
       NOTREACHED() << "Unhandled action " << it->second;
   }
   return false;
 }
 
 void AcceleratorController::SwitchToWindow(int window) {
   const LauncherItems& items =
       Shell::GetInstance()->launcher()->model()->items();
@@ skipping 20 matching lines @@
     // Then set this one as active.
     Shell::GetInstance()->launcher()->ActivateLauncherItem(found_index);
   }
 }
 
 bool AcceleratorController::CanHandleAccelerators() const {
   return true;
 }
 
 }  // namespace ash