Fix the wrong relative coordinates from EventDispatch

ui/aura/window_event_dispatcher.cc

Index: ui/aura/window_event_dispatcher.cc
diff --git a/ui/aura/window_event_dispatcher.cc b/ui/aura/window_event_dispatcher.cc
index 3313d137bca352262566c28ebc10e114f02732d5..a0a140d1565c0cca0b2c74ac0ce004ba0a1620ee 100644
--- a/ui/aura/window_event_dispatcher.cc
+++ b/ui/aura/window_event_dispatcher.cc
@@ -149,11 +149,11 @@ void WindowEventDispatcher::DispatchGestureEvent(ui::GestureEvent* event) {
   }
 }
 
-DispatchDetails WindowEventDispatcher::DispatchMouseExitAtPoint(
+DispatchDetails WindowEventDispatcher::DispatchMouseExitAtPoint(Window* window,
     const gfx::Point& point) {
   ui::MouseEvent event(ui::ET_MOUSE_EXITED, point, point, ui::EF_NONE,
                        ui::EF_NONE);
-  return DispatchMouseEnterOrExit(event, ui::ET_MOUSE_EXITED);
+  return DispatchMouseEnterOrExit(window, event, ui::ET_MOUSE_EXITED);
 }
 
 void WindowEventDispatcher::ProcessedTouchEvent(ui::TouchEvent* event,
@@ -242,13 +242,15 @@ void WindowEventDispatcher::DispatchMouseExitToHidingWindow(Window* window) {
   gfx::Point last_mouse_location = GetLastMouseLocationInRoot();
   if (window->Contains(mouse_moved_handler_) &&
       window->ContainsPointInRoot(last_mouse_location)) {
-    DispatchDetails details = DispatchMouseExitAtPoint(last_mouse_location);
+    DispatchDetails details = DispatchMouseExitAtPoint(window,
+        last_mouse_location);
     if (details.dispatcher_destroyed)
       return;
   }
 }
 
 ui::EventDispatchDetails WindowEventDispatcher::DispatchMouseEnterOrExit(
+    Window* target,
     const ui::MouseEvent& event,
     ui::EventType type) {
   if (event.type() != ui::ET_MOUSE_CAPTURE_CHANGED &&
@@ -265,7 +267,8 @@ ui::EventDispatchDetails WindowEventDispatcher::DispatchMouseEnterOrExit(
   // system), or a synthetic event created in root-window (in which case, the
   // event's target will be NULL, and the event will be in the root-window's
   // coordinate system.
-  aura::Window* target = static_cast<Window*>(event.target());
+  if (event.target())

[sky, 2014/12/18 20:29:29]

Why do we need both the supplied target and event.target()? 
WindowEventDispatcher::DispatchMouseExitAtPoint is creating an event, can't it
set the window on the target?

[Miyoung Shin, 2014/12/19 14:39:42]

You're right.
I think I can use only target to consider both of DispatchMouseExitAtPoint and
PreDispatchMouseEvent. I will change.

[sky, 2015/01/05 17:23:02]

You didn't quite end up with what I was suggesting. I'm wondering if when
WindowEventDispatcher::DispatchMouseExitAtPoint creates the MouseEvent it can
set the target on the MouseEvent. I believe this would mean you don't need to
supply an additional target.

[Miyoung Shin, 2015/01/06 13:20:44]

Sorry, I don't understand your point yet and why you mentioned
DispatchMouseExitAtPoint.
I can set the target on the MouseEvent in 
WindowEventDispatcher::DispatchMouseExitAtPoint.
But I have no idea how to set the target in
WindowEventDispatcher::PreDispatchMouseEvent that has originally the issue
without supplying an additional target.
Do you mean I should add setTarget method in Event class for
PreDispatchMouseEvent?
If I'm wrong, please correct me.

+    target = static_cast<Window*>(event.target());
   if (!target)
     target = window();
   ui::MouseEvent translated_event(event,
@@ -404,7 +407,7 @@ void WindowEventDispatcher::OnOtherRootGotCapture() {
     // Dispatch a mouse exit to reset any state associated with hover. This is
     // important when going from no window having capture to a window having
     // capture because we do not dispatch ET_MOUSE_CAPTURE_CHANGED in this case.
-    DispatchDetails details = DispatchMouseExitAtPoint(
+    DispatchDetails details = DispatchMouseExitAtPoint(NULL,

[sky, 2014/12/18 20:29:29]

nullptr

[Miyoung Shin, 2014/12/19 14:39:42]

ok

         GetLastMouseLocationInRoot());
     if (details.dispatcher_destroyed)
       return;
@@ -785,7 +788,7 @@ void WindowEventDispatcher::PreDispatchMouseEvent(Window* target,
     case ui::ET_MOUSE_EXITED:
       if (!target || target == window()) {
         DispatchDetails details =
-            DispatchMouseEnterOrExit(*event, ui::ET_MOUSE_EXITED);
+            DispatchMouseEnterOrExit(target, *event, ui::ET_MOUSE_EXITED);
         if (details.dispatcher_destroyed) {
           event->SetHandled();
           return;
@@ -802,7 +805,7 @@ void WindowEventDispatcher::PreDispatchMouseEvent(Window* target,
         WindowTracker live_window;
         live_window.Add(target);
         DispatchDetails details =
-            DispatchMouseEnterOrExit(*event, ui::ET_MOUSE_EXITED);
+            DispatchMouseEnterOrExit(target, *event, ui::ET_MOUSE_EXITED);
         if (details.dispatcher_destroyed) {
           event->SetHandled();
           return;
@@ -821,7 +824,8 @@ void WindowEventDispatcher::PreDispatchMouseEvent(Window* target,
         live_window.Remove(target);
 
         mouse_moved_handler_ = target;
-        details = DispatchMouseEnterOrExit(*event, ui::ET_MOUSE_ENTERED);
+        details = DispatchMouseEnterOrExit(target, *event,
+            ui::ET_MOUSE_ENTERED);
         if (details.dispatcher_destroyed || details.target_destroyed) {
           event->SetHandled();
           return;