Fix leak of HID transfers and handles on Windows.

Fix leak of HID transfers and handles on Windows.

This patch reworks the management of PendingHidTransfer objects in the
Windows HID backend so that they are no longer reference counted. It
then adds logic to free all transfers when the device handle is closed
in order to break the circular reference between transfers and the
connection object.

BUG=618213
Committed: https://crrev.com/e62aec48485940cb4e214dd8f6845ebc89f20dee
Cr-Commit-Position: refs/heads/master@{#438374}

[Reilly Grant (use Gerrit), 2016-12-13 01:09:55]

The CQ bit was checked by reillyg@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 01:10:42]

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

[Reilly Grant (use Gerrit), 2016-12-13 01:12:34]

reillyg@chromium.org changed reviewers:
+ juncai@chromium.org

[Reilly Grant (use Gerrit), 2016-12-13 01:12:34]

Please take a look.

[commit-bot: I haz the power, 2016-12-13 02:31:48]

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

[commit-bot: I haz the power, 2016-12-13 02:31:49]

Dry run: This issue passed the CQ dry run.

[juncai, 2016-12-14 00:54:24]

LGTM.

[Reilly Grant (use Gerrit), 2016-12-14 01:04:30]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2016-12-14 01:05:18]

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, 2016-12-14 01:11:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481677470521670, "parent_rev":
"1b461fc86446caaa622e07276e62cfa6e4413b70", "commit_rev":
"df65e73e60115e9e06cb3aeba0ff7e26a3877beb"}

[commit-bot: I haz the power, 2016-12-14 01:12:51]

Description was changed from

==========
Fix leak of HID transfers and handles on Windows.

This patch reworks the management of PendingHidTransfer objects in the
Windows HID backend so that they are no longer reference counted. It
then adds logic to free all transfers when the device handle is closed
in order to break the circular reference between transfers and the
connection object.

BUG=618213
==========

to

==========
Fix leak of HID transfers and handles on Windows.

This patch reworks the management of PendingHidTransfer objects in the
Windows HID backend so that they are no longer reference counted. It
then adds logic to free all transfers when the device handle is closed
in order to break the circular reference between transfers and the
connection object.

BUG=618213

Review-Url: https://codereview.chromium.org/2573683002
==========

[commit-bot: I haz the power, 2016-12-14 01:12:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-14 01:15:30]

Description was changed from

==========
Fix leak of HID transfers and handles on Windows.

This patch reworks the management of PendingHidTransfer objects in the
Windows HID backend so that they are no longer reference counted. It
then adds logic to free all transfers when the device handle is closed
in order to break the circular reference between transfers and the
connection object.

BUG=618213

Review-Url: https://codereview.chromium.org/2573683002
==========

to

==========
Fix leak of HID transfers and handles on Windows.

This patch reworks the management of PendingHidTransfer objects in the
Windows HID backend so that they are no longer reference counted. It
then adds logic to free all transfers when the device handle is closed
in order to break the circular reference between transfers and the
connection object.

BUG=618213

Committed: https://crrev.com/e62aec48485940cb4e214dd8f6845ebc89f20dee
Cr-Commit-Position: refs/heads/master@{#438374}
==========

[commit-bot: I haz the power, 2016-12-14 01:15:31]

Patchset 1 (id:??) landed as
https://crrev.com/e62aec48485940cb4e214dd8f6845ebc89f20dee
Cr-Commit-Position: refs/heads/master@{#438374}