Move cursor when it's hidden so that UX can update correctly

Move cursor when it's hidden so that UX can update correctly


BUG=136057
TEST=None

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=151705

[oshima, 2012-08-06 03:24:24]

https://chromiumcodereview.appspot.com/10829180/diff/2001/ui/aura/root_window.cc
File ui/aura/root_window.cc (right):

https://chromiumcodereview.appspot.com/10829180/diff/2001/ui/aura/root_window...
ui/aura/root_window.cc:240: }
I think it's better to change last_mouse_location to somewhere really off when
the mouse cursor is not visible,
and simply call PostMouseMoveEventAfterWindowChange because
1) last_mouse_location is used in other places and it may cause similar issue.
2) there may be an posted task that sends synthesized mouse move, which can
override the call above.

Ideally, we should move the mouse AND ignore all mouse events after that, but
that's probably outside of this CL's scope.

https://chromiumcodereview.appspot.com/10829180/diff/2001/ui/aura/root_window...
ui/aura/root_window.cc:1032: GetLastMouseLocationInRoot() : gfx::Point(-1, -1));
-1, -1 may still overlap a views' component. I'd suggest to use something like
(-10000, -10000)

[sky, 2012-08-14 16:01:00]

https://chromiumcodereview.appspot.com/10829180/diff/12001/ui/aura/env.cc
File ui/aura/env.cc (right):

https://chromiumcodereview.appspot.com/10829180/diff/12001/ui/aura/env.cc#new...
ui/aura/env.cc:81: last_mouse_location_ = gfx::Point(-10000, -10000);
Won't this cause all sorts of weirdness if the hide occurs during a mouse drag
or something?

[sky, 2012-08-14 23:56:59]

LGTM

[commit-bot: I haz the power, 2012-08-15 00:16:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davemoore@chromium.org/10829180/12001

[commit-bot: I haz the power, 2012-08-15 02:17:56]

Try job failure for 10829180-12001 (retry) on linux_rel for step
"interactive_ui_tests".
It's a second try, previously, steps "interactive_ui_tests, browser_tests"
failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...