Fix crash of OmniboxViewViewsTest.SelectAllOnClick on Linux Aura

ui/aura/test/ui_controls_factory_aurax11.cc

 // Copyright 2013 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 <X11/keysym.h>
 #include <X11/Xlib.h>
 
 // X macro fail.
 #if defined(RootWindow)
 #undef RootWindow
@@ skipping 121 matching lines @@
     DCHECK(!xevent.xkey.state);
     RunClosureAfterAllPendingUIEvents(closure);
     return true;
   }
 
   // Simulate a mouse move. (x,y) are absolute screen coordinates.
   virtual bool SendMouseMove(long x, long y) OVERRIDE {
     return SendMouseMoveNotifyWhenDone(x, y, base::Closure());
   }
   virtual bool SendMouseMoveNotifyWhenDone(
-      long x,
-      long y,
+      long root_x,

[sadrul, 2014/02/14 19:18:10]

The desktop version calls this screen_x, screen_y (and the doc for
UIControlsAura also claims x,y are in absolute screen coordinates). So let's
call these screen_x|y and use the ScreenPositionClient (although I suppose in
the case of chromeos it doesn't matter because the window is positioned at 0,0)?

[pkotwicz, 2014/02/14 19:57:18]

Root window coordinates are passed in. The documentation is wrong. Notice
GetUIControlsAt() in ui_controls_factory_ash.cc I agree that this is confusing.
I will fix this as part of a separate CL. (Separate CL because I need to test
this on Windows too)

+      long root_y,
       const base::Closure& closure) OVERRIDE {
-    XEvent xevent = {0};
-    XMotionEvent* xmotion = &xevent.xmotion;
-    xmotion->type = MotionNotify;
-    gfx::Point point = ui::ConvertPointToPixel(
-        root_window_->window()->layer(),
-        gfx::Point(static_cast<int>(x), static_cast<int>(y)));
-    xmotion->x = point.x();
-    xmotion->y = point.y();
-    xmotion->state = button_down_mask;
-    xmotion->same_screen = True;
-    // RootWindow will take care of other necessary fields.
-    root_window_->host()->PostNativeEvent(&xevent);
+    gfx::Point root_location(static_cast<int>(root_x),
+                             static_cast<int>(root_y));
+
+    gfx::Point root_current_location;
+    root_window_->host()->QueryMouseLocation(&root_current_location);
+    root_window_->host()->ConvertPointFromHost(&root_current_location);
+
+    if (root_location != root_current_location && button_down_mask == 0) {
+      // Move the cursor because EnterNotify/LeaveNotify are generated with the
+      // current mouse position as a result of XGrabPointer()
+      root_window_->MoveCursorTo(root_location);
+    } else {
+      XEvent xevent = {0};
+      XMotionEvent* xmotion = &xevent.xmotion;
+      xmotion->type = MotionNotify;
+      xmotion->x = root_location.x();
+      xmotion->y = root_location.y();
+      xmotion->state = button_down_mask;
+      xmotion->same_screen = True;
+      // RootWindow will take care of other necessary fields.
+      root_window_->host()->PostNativeEvent(&xevent);
+    }
     RunClosureAfterAllPendingUIEvents(closure);
     return true;
   }
   virtual bool SendMouseEvents(MouseButton type, int state) OVERRIDE {
     return SendMouseEventsNotifyWhenDone(type, state, base::Closure());
   }
   virtual bool SendMouseEventsNotifyWhenDone(
       MouseButton type,
       int state,
       const base::Closure& closure) OVERRIDE {
@@ skipping 81 matching lines @@
 };
 
 }  // namespace
 
 UIControlsAura* CreateUIControlsAura(aura::RootWindow* root_window) {
   return new UIControlsX11(root_window);
 }
 
 }  // namespace test
 }  // namespace aura