MouseEventManager: reset drag state upon detaching.

MouseEventManager: reset drag state upon detaching.

If an ongoing drag detaches the frame of the dragged element, have
the MouseEventManager reset the drag state while handling the detachment
from the frame. Otherwise the DragState will retain a reference to
the drag source (and its document) for too long, triggering a reported
leak on shutdown.

R=
BUG=703057
Review-Url: https://codereview.chromium.org/2762613002
Cr-Commit-Position: refs/heads/master@{#458116}
Committed: https://chromium.googlesource.com/chromium/src/+/f6108b39ff0fa823e85bada2e56dbf89d4f9acad

[sof, 2017-03-20 09:39:05]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-20 09:39:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sof, 2017-03-20 10:52:39]

sigbjornf@opera.com changed reviewers:
+ dcheng@chromium.org

[sof, 2017-03-20 10:52:39]

please take a look.

[commit-bot: I haz the power, 2017-03-20 12:01:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-20 12:01:11]

Dry run: This issue passed the CQ dry run.

[dcheng, 2017-03-20 17:43:51]

lgtm

[dcheng, 2017-03-20 17:43:59]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2017-03-20 17:44:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-20 17:50:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490031839918980, "parent_rev":
"a17c2172f68e4f4cb3680fa483270c183e8e7e3e", "commit_rev":
"f6108b39ff0fa823e85bada2e56dbf89d4f9acad"}

[commit-bot: I haz the power, 2017-03-20 17:50:53]

Description was changed from

==========
MouseEventManager: reset drag state upon detaching.

If an ongoing drag detaches the frame of the dragged element, have
the MouseEventManager reset the drag state while handling the detachment
from the frame. Otherwise the DragState will retain a reference to
the drag source (and its document) for too long, triggering a reported
leak on shutdown.

R=
BUG=703057
==========

to

==========
MouseEventManager: reset drag state upon detaching.

If an ongoing drag detaches the frame of the dragged element, have
the MouseEventManager reset the drag state while handling the detachment
from the frame. Otherwise the DragState will retain a reference to
the drag source (and its document) for too long, triggering a reported
leak on shutdown.

R=
BUG=703057

Review-Url: https://codereview.chromium.org/2762613002
Cr-Commit-Position: refs/heads/master@{#458116}
Committed:
https://chromium.googlesource.com/chromium/src/+/f6108b39ff0fa823e85bada2e56d...
==========

[commit-bot: I haz the power, 2017-03-20 17:50:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f6108b39ff0fa823e85bada2e56d...