Check callback is run in QuicChromiumClientSession's destructor

Check callback is run in QuicChromiumClientSession's destructor

Add a DCHECK to make sure |callback_| has been run in
QuicChromiumClientSession's destructor.

BUG=704949
Review-Url: https://codereview.chromium.org/2797443002
Cr-Commit-Position: refs/heads/master@{#461558}
Committed: https://chromium.googlesource.com/chromium/src/+/e23ca12cab688382ed5a3e3350885bed0b9d39b4

[xunjieli, 2017-04-03 15:10:31]

Description was changed from

==========
Check callback is run in QuicChromiumClientSession's destructor

Add a DCHECK to make sure |callback_| has been run in
QuicChromiumClientSession's destructor.

BUG=704949
==========

to

==========
Check callback is run in QuicChromiumClientSession's destructor

Add a DCHECK to make sure |callback_| has been run in
QuicChromiumClientSession's destructor.

BUG=704949
==========

[xunjieli, 2017-04-03 15:10:31]

xunjieli@chromium.org changed reviewers:
+ zhongyi@chromium.org

[xunjieli, 2017-04-03 15:10:35]

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

[commit-bot: I haz the power, 2017-04-03 15:10:57]

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-04-03 16:21:37]

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

[commit-bot: I haz the power, 2017-04-03 16:21:38]

Dry run: This issue passed the CQ dry run.

[Zhongyi Shi, 2017-04-03 22:15:50]

The CQ bit was checked by zhongyi@chromium.org

[Zhongyi Shi, 2017-04-03 22:15:50]

lgtm

[commit-bot: I haz the power, 2017-04-03 22:16:45]

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-04-03 22:23:35]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491257750341650, "parent_rev":
"e385704a77bfd6b5fd665bf860ae22beae237802", "commit_rev":
"e23ca12cab688382ed5a3e3350885bed0b9d39b4"}

[commit-bot: I haz the power, 2017-04-03 22:24:33]

Description was changed from

==========
Check callback is run in QuicChromiumClientSession's destructor

Add a DCHECK to make sure |callback_| has been run in
QuicChromiumClientSession's destructor.

BUG=704949
==========

to

==========
Check callback is run in QuicChromiumClientSession's destructor

Add a DCHECK to make sure |callback_| has been run in
QuicChromiumClientSession's destructor.

BUG=704949

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

[commit-bot: I haz the power, 2017-04-03 22:24:35]

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