aura: Fix a couple of drag drop issues:

aura: Fix a couple of drag drop issues:
1. Only DragDropController should be allowed to set the cursor while a drag-drop
session is in progress.
2. For some wierd reason I was cancelling drag-drop on MOUSE_EXITED event. That
is clearly wrong.

BUG=138694
TEST=manual


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149670

[sky, 2012-08-02 16:26:41]

https://chromiumcodereview.appspot.com/10825143/diff/1002/ash/drag_drop/drag_...
File ash/drag_drop/drag_drop_controller.cc (right):

https://chromiumcodereview.appspot.com/10825143/diff/1002/ash/drag_drop/drag_...
ash/drag_drop/drag_drop_controller.cc:37:
ash::Shell::GetInstance()->cursor_manager()->UnlockCursor();
Unlock/Lock decrement/increment a count. So, it's entirely possible this won't
set the cursor.

[sky, 2012-08-02 17:27:36]

Also, I'm not sure you always want to force it. For example, if the
drag is the result of touch do you want to change the cursor?

  -Scott

On Thu, Aug 2, 2012 at 9:26 AM,  <sky@chromium.org> wrote:
>
>
https://chromiumcodereview.appspot.com/10825143/diff/1002/ash/drag_drop/drag_...
> File ash/drag_drop/drag_drop_controller.cc (right):
>
>
https://chromiumcodereview.appspot.com/10825143/diff/1002/ash/drag_drop/drag_...
> ash/drag_drop/drag_drop_controller.cc:37:
>
> ash::Shell::GetInstance()->cursor_manager()->UnlockCursor();
> Unlock/Lock decrement/increment a count. So, it's entirely possible this
> won't set the cursor.
>
> https://chromiumcodereview.appspot.com/10825143/

[varunjain, 2012-08-02 17:29:52]

https://chromiumcodereview.appspot.com/10825143/diff/1002/ash/drag_drop/drag_...
File ash/drag_drop/drag_drop_controller.cc (right):

https://chromiumcodereview.appspot.com/10825143/diff/1002/ash/drag_drop/drag_...
ash/drag_drop/drag_drop_controller.cc:37:
ash::Shell::GetInstance()->cursor_manager()->UnlockCursor();
On 2012/08/02 16:26:42, sky wrote:
> Unlock/Lock decrement/increment a count. So, it's entirely possible this won't
> set the cursor.

I have moved the cursor locking to the RootWindow.
I wish there was some nicer way to use the cursor manager's locking
functionality :(
Do you have any other idea?

[sky, 2012-08-02 17:38:34]

LGTM

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/varunjain@chromium.org/10825143/6001

[commit-bot: I haz the power, 2012-08-02 19:19:59]

Change committed as 149670