Adding ThreadChecker validation to d'tors for client classes

Adding ThreadChecker validation to d'tors for client classes

There was an incorrect assumption about how ThreadChecker worked which lead
to several classes not having the d'tor guarantees they expected.  The
assumption was that a ThreadChecker would DCHECK if it was not destroyed on
the same thread it was bound to.  This is not correct so I am adding DCHECKs
to the d'tors of the remoting classes which were missed.

BUG=715633
Review-Url: https://codereview.chromium.org/2886453004
Cr-Commit-Position: refs/heads/master@{#471839}
Committed: https://chromium.googlesource.com/chromium/src/+/4d4db58f0615aa162f870821f1bba6815a8b8dd3

[joedow, 2017-05-15 16:03:47]

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

[commit-bot: I haz the power, 2017-05-15 16:04:26]

Dry run: 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-05-15 16:41:02]

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

[commit-bot: I haz the power, 2017-05-15 16:41:04]

Dry run: This issue passed the CQ dry run.

[joedow, 2017-05-15 16:55:57]

joedow@chromium.org changed reviewers:
+ yuweih@chromium.org

[joedow, 2017-05-15 16:55:58]

PTAL!

[Yuwei, 2017-05-15 18:23:09]

lgtm

[joedow, 2017-05-15 18:23:54]

The CQ bit was checked by joedow@chromium.org

[commit-bot: I haz the power, 2017-05-15 18:24:56]

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-05-15 18:34:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494872634168070, "parent_rev":
"6c47dc6c68d89edb553cb66eedc823e011f46b68", "commit_rev":
"4d4db58f0615aa162f870821f1bba6815a8b8dd3"}

[commit-bot: I haz the power, 2017-05-15 18:35:03]

Description was changed from

==========
Adding ThreadChecker validation to d'tors for client classes

There was an incorrect assumption about how ThreadChecker worked which lead
to several classes not having the d'tor guarantees they expected.  The
assumption was that a ThreadChecker would DCHECK if it was not destroyed on
the same thread it was bound to.  This is not correct so I am adding DCHECKs
to the d'tors of the remoting classes which were missed.

BUG=715633
==========

to

==========
Adding ThreadChecker validation to d'tors for client classes

There was an incorrect assumption about how ThreadChecker worked which lead
to several classes not having the d'tor guarantees they expected.  The
assumption was that a ThreadChecker would DCHECK if it was not destroyed on
the same thread it was bound to.  This is not correct so I am adding DCHECKs
to the d'tors of the remoting classes which were missed.

BUG=715633

Review-Url: https://codereview.chromium.org/2886453004
Cr-Commit-Position: refs/heads/master@{#471839}
Committed:
https://chromium.googlesource.com/chromium/src/+/4d4db58f0615aa162f870821f1bb...
==========

[commit-bot: I haz the power, 2017-05-15 18:35:04]

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