Fix the wrong relative coordinates from EventDispatch

ui/aura/window_event_dispatcher_unittest.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 "ui/aura/window_event_dispatcher.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/run_loop.h"
@@ skipping 789 matching lines @@
   DispatchEventUsingWindowDispatcher(&touch_moved_event3);
   DispatchEventUsingWindowDispatcher(&touch_released_event);
   EXPECT_EQ("TOUCH_MOVED TOUCH_RELEASED GESTURE_TAP GESTURE_END",
             EventTypesToString(recorder.events()));
   recorder.Reset();
   host()->dispatcher()->ReleasePointerMoves();
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.events().empty());
 }
 
+// Tests that mouse move event has a right location
+// when there isn't the target window
+TEST_F(WindowEventDispatcherTest, MouseEventWithoutTargetWindow) {
+  EventFilterRecorder recorder_first;
+  EventFilterRecorder recorder_second;
+
+  test::TestWindowDelegate delegate;
+  scoped_ptr<aura::Window> window_first(CreateTestWindowWithDelegate(
+      &delegate, 1, gfx::Rect(20, 10, 10, 20), root_window()));
+  window_first->Show();
+  window_first->AddPreTargetHandler(&recorder_first);
+
+  scoped_ptr<aura::Window> window_second(CreateTestWindowWithDelegate(
+      &delegate, 2, gfx::Rect(20, 30, 10, 20), root_window()));
+  window_second->Show();
+  window_second->AddPreTargetHandler(&recorder_second);
+
+  const gfx::Point event_location(22, 33);
+  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, event_location,
+                        event_location, 0, 0);
+  DispatchEventUsingWindowDispatcher(&mouse);
+
+  EXPECT_TRUE(recorder_first.events().empty());
+  EXPECT_FALSE(recorder_second.events().empty());

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

Doesn't 834 handle this assertion?

+  EXPECT_EQ("MOUSE_ENTERED MOUSE_MOVED",
+            EventTypesToString(recorder_second.events()));
+  EXPECT_EQ(gfx::Point(2, 3).ToString(),

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

You need to assert mouse_locations() is non-empty, otherwise if it is empty this
line crashes.

+            recorder_second.mouse_locations()[0].ToString());
+}
+
 // Verifies that a direct call to ProcessedTouchEvent() with a
 // TOUCH_PRESSED event does not cause a crash.
 TEST_F(WindowEventDispatcherTest, CallToProcessedTouchEvent) {
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(50, 50, 100, 100), root_window()));
 
   ui::TouchEvent touch(
       ui::ET_TOUCH_PRESSED, gfx::Point(10, 10), 1, ui::EventTimeForNow());
   host()->dispatcher()->ProcessedTouchEvent(
@@ skipping 1552 matching lines @@
   ASSERT_EQ(1u, recorder.touch_locations().size());
   EXPECT_EQ(gfx::Point(kX - kWindowOffset, kY - kWindowOffset).ToString(),
             recorder.touch_locations()[0].ToString());
 
   ASSERT_EQ(2u, recorder.gesture_locations().size());
   EXPECT_EQ(gfx::Point(kX - kWindowOffset, kY - kWindowOffset).ToString(),
             recorder.gesture_locations()[0].ToString());
 }
 
 }  // namespace aura