Avoid changing |drag_reverted_| if the drag was already completed on ChromeOS

Avoid changing |drag_reverted_| if the drag was already completed on ChromeOS

This is part #2 of https://codereview.chromium.org/354343002/

BUG=None
TEST=None

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

[pkotwicz, 2014-07-04 15:27:45]

Scott, can you please take a look?

I mostly want your opinion on never handling the ET_MOUSE_CAPTURE_CHANGED event.
The handling was added by https://chromiumcodereview.appspot.com/9455063

The code was made into a no-op as a result of my bad advice.
https://codereview.chromium.org/144193002

I can't think of a case where running View::OnMouseCaptureLost() on an invisible
widget would be problematic.

[varkha, 2014-07-08 14:48:37]

https://codereview.chromium.org/345133005/diff/90001/ash/wm/toplevel_window_e...
File ash/wm/toplevel_window_event_handler.h (right):

https://codereview.chromium.org/345133005/diff/90001/ash/wm/toplevel_window_e...
ash/wm/toplevel_window_event_handler.h:77: // completed.
nit: s/"completed"/"completed or reverted" since it seems to return true in both
cases?

[sky, 2014-07-09 20:27:02]

https://codereview.chromium.org/345133005/diff/130001/ash/wm/toplevel_window_...
File ash/wm/toplevel_window_event_handler.cc (left):

https://codereview.chromium.org/345133005/diff/130001/ash/wm/toplevel_window_...
ash/wm/toplevel_window_event_handler.cc:509: event->SetHandled();
Why is marking the event has handled no longer necessary?

[pkotwicz, 2014-07-10 00:23:52]

https://codereview.chromium.org/345133005/diff/130001/ash/wm/toplevel_window_...
File ash/wm/toplevel_window_event_handler.cc (left):

https://codereview.chromium.org/345133005/diff/130001/ash/wm/toplevel_window_...
ash/wm/toplevel_window_event_handler.cc:509: event->SetHandled();
I couldn't find a case where not handling ui::ET_MOUSE_CAPTURE_CHANGED caused
problems.

This if statement has been a no-op since M34 and there have not been any bug
reports wrt to it being a no-op.

[sky, 2014-07-10 16:13:36]

Ok, LGTM

[pkotwicz, 2014-07-10 17:03:32]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2014-07-10 17:05:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/345133005/190001

[commit-bot: I haz the power, 2014-07-10 19:21:08]

Change committed as 282392