Use root window's coordinates when updating cursor in Aura

content/browser/renderer_host/render_widget_host_view_aura.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 "content/browser/renderer_host/render_widget_host_view_aura.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/command_line.h"
@@ skipping 3169 matching lines @@
 
 void RenderWidgetHostViewAura::UpdateCursorIfOverSelf() {
   const gfx::Point screen_point =
       gfx::Screen::GetScreenFor(GetNativeView())->GetCursorScreenPoint();
   aura::RootWindow* root_window = window_->GetRootWindow();
   if (!root_window)
     return;
 
   gfx::Rect screen_rect = GetViewBounds();
   gfx::Point local_point = screen_point;
   local_point.Offset(-screen_rect.x(), -screen_rect.y());

[sadrul, 2013/10/01 14:33:36]

I think this is incorrect. This should use ScreenPositionClient to convert
|screen_point| to |local_point|.

[tdanderson, 2013/10/17 18:23:47]

Done.

 
 #if defined(OS_WIN)
   // If there's another toplevel window above us at this point (for example a
   // menu), we don't want to update the cursor.
   POINT windows_point = { screen_point.x(), screen_point.y() };
-  if (root_window->GetAcceleratedWidget() != ::WindowFromPoint(windows_point))
+  if (root_window->GetEventHandlerForPoint(local_point) != window_ ||
+      root_window->GetAcceleratedWidget() != ::WindowFromPoint(windows_point))
+    return;
+#else
+  if (root_window->GetEventHandlerForPoint(screen_point) != window_)

[sadrul, 2013/10/01 14:33:36]

Window::GetEventHandlerForPoint() expects the location to be in the Window's
local coordinates. So both of these calls are incorrect.

[tdanderson, 2013/10/17 18:23:47]

Done.

     return;
 #endif
-  if (root_window->GetEventHandlerForPoint(local_point) != window_)
-    return;
 
   gfx::NativeCursor cursor = current_cursor_.GetNativeCursor();
   // Do not show loading cursor when the cursor is currently hidden.
   if (is_loading_ && cursor != ui::kCursorNone)
     cursor = ui::kCursorPointer;
 
   aura::client::CursorClient* cursor_client =
       aura::client::GetCursorClient(root_window);
   if (cursor_client) {
     cursor_client->SetCursor(cursor);
@@ skipping 166 matching lines @@
     RenderWidgetHost* widget) {
   return new RenderWidgetHostViewAura(widget);
 }
 
 // static
 void RenderWidgetHostViewPort::GetDefaultScreenInfo(WebScreenInfo* results) {
   GetScreenInfoForWindow(results, NULL);
 }
 
 }  // namespace content