Do not let MagnificationController move cursor on login screen when cursor is invisible.

ash/magnifier/magnification_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/magnifier/magnification_controller.h"
 
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ui/aura/client/cursor_client.h"
@@ skipping 89 matching lines @@
   //  - Unzoom the current root_window.
   //  - Zoom the given new root_window |new_root_window|.
   //  - Switch the target window from current window to |new_root_window|.
   void SwitchTargetRootWindow(aura::RootWindow* new_root_window);
 
   // Returns if the magnification scale is 1.0 or not (larger then 1.0).
   bool IsMagnified() const;
 
   // Returns the default scale which depends on the login status.
   float GetDefaultZoomScale() const {
+    // TODO(yoshiki,mazda): Do not use SystemTrayDelegate to get the user login
+    // status (http://crbug.com/163170).
     user::LoginStatus login = Shell::GetInstance()->tray_delegate() ?
         Shell::GetInstance()->tray_delegate()->GetUserLoginStatus() :
         user::LOGGED_IN_NONE;
 
     // On login screen, don't magnify the screen by default.
     if (login == user::LOGGED_IN_NONE)
       return kNonMagnifiedScale;
 
     return kInitialMagnifiedScale;
   }
@@ skipping 32 matching lines @@
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // MagnificationControllerImpl:
 
 MagnificationControllerImpl::MagnificationControllerImpl()
     : root_window_(ash::Shell::GetPrimaryRootWindow()),
       is_on_animation_(false),
       is_enabled_(false),
       move_cursor_after_animation_(false),
-      scale_(std::numeric_limits<double>::min()) {
+      scale_(kNonMagnifiedScale) {
   Shell::GetInstance()->AddPreTargetHandler(this);
 }
 
 MagnificationControllerImpl::~MagnificationControllerImpl() {
   Shell::GetInstance()->RemovePreTargetHandler(this);
 }
 
 void MagnificationControllerImpl::RedrawKeepingMousePosition(
     float scale, bool animate) {
   gfx::Point mouse_in_root = root_window_->GetLastMouseLocationInRoot();
@@ skipping 171 matching lines @@
       if (x_diff != 0 || y_diff != 0)
         AfterAnimationMoveCursorTo(mouse);
     }
   }
 }
 
 void MagnificationControllerImpl::AfterAnimationMoveCursorTo(
     const gfx::Point& location) {
   aura::client::CursorClient* cursor_client =
       aura::client::GetCursorClient(root_window_);
-  if (cursor_client)
+  if (cursor_client) {
+    // When cursor is invisible, do not move or show the cursor after the
+    // animation.
+    if (!cursor_client->IsCursorVisible())
+      return;
     cursor_client->ShowCursor(false);
+  }
   move_cursor_after_animation_ = true;
   position_after_animation_ = location;
 }
 
 gfx::Size MagnificationControllerImpl::GetHostSizeDIP() const {
   return ui::ConvertSizeToDIP(root_window_->layer(),
                               root_window_->GetHostSize());
 }
 
 gfx::RectF MagnificationControllerImpl::GetWindowRectDIP(float scale) const {
@@ skipping 24 matching lines @@
   DCHECK(kNonMagnifiedScale <= *scale && *scale <= kMaxMagnifiedScale);
 }
 
 void MagnificationControllerImpl::OnImplicitAnimationsCompleted() {
   if (!is_on_animation_)
     return;
 
   if (move_cursor_after_animation_) {
     root_window_->MoveCursorTo(position_after_animation_);
     move_cursor_after_animation_ = false;
+
+    aura::client::CursorClient* cursor_client =
+        aura::client::GetCursorClient(root_window_);
+    if (cursor_client)
+      cursor_client->ShowCursor(true);
   }
 
-  aura::client::CursorClient* cursor_client =
-      aura::client::GetCursorClient(root_window_);
-  if (cursor_client)
-    cursor_client->ShowCursor(true);
-
   is_on_animation_ = false;
 }
 
 void MagnificationControllerImpl::SwitchTargetRootWindow(
     aura::RootWindow* new_root_window) {
   if (new_root_window == root_window_)
     return;
 
   float scale = GetScale();
 
@@ skipping 109 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // MagnificationController:
 
 // static
 MagnificationController* MagnificationController::CreateInstance() {
   return new MagnificationControllerImpl();
 }
 
 }  // namespace ash