Add CHECKs to make sure ThreadableLoader has not been created multiple times

Add CHECKs to make sure ThreadableLoader has not been created multiple times

A ThreadableLoader must be completed before it's destructed. We need to forbid
to create ThreadableLoader multiple times in some classes to ensure that.
This CL adds CHECKs to make sure that.

BUG=667254
Review-Url: https://codereview.chromium.org/2629823002
Cr-Commit-Position: refs/heads/master@{#443506}
Committed: https://chromium.googlesource.com/chromium/src/+/9cdca0764fe0ab606d904f8e2bd6eb5117773a97

[yhirano, 2017-01-13 04:06:17]

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

[commit-bot: I haz the power, 2017-01-13 04:06:42]

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

[yhirano, 2017-01-13 04:12:25]

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

[commit-bot: I haz the power, 2017-01-13 04:12:50]

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

[yhirano, 2017-01-13 04:13:19]

yhirano@chromium.org changed reviewers:
+ tyoshino@chromium.org

[commit-bot: I haz the power, 2017-01-13 05:55:32]

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

[commit-bot: I haz the power, 2017-01-13 05:55:33]

Dry run: This issue passed the CQ dry run.

[tyoshino (SeeGerritForStatus), 2017-01-13 06:04:42]

lgtm

[yhirano, 2017-01-13 06:09:07]

Description was changed from

==========
Add CHECKs to make sure ThreadableLoader has not been created multiple times

A ThreadableLoader must be completed before it's destructed. We need to forbid
to create ThreadableLoader multiple times in some classes to ensure that.
This CL adds CHECKs to make sure that.

BUG=667254
==========

to

==========
Add CHECKs to make sure ThreadableLoader has not been created multiple times

A ThreadableLoader must be completed before it's destructed. We need to forbid
to create ThreadableLoader multiple times in some classes to ensure that.
This CL adds CHECKs to make sure that.

BUG=667254
==========

[yhirano, 2017-01-13 06:09:07]

yhirano@chromium.org changed reviewers:
+ haraken@chromium.org

[yhirano, 2017-01-13 06:09:20]

+haraken@ for owner review.

[haraken, 2017-01-13 06:10:14]

LGTM

[yhirano, 2017-01-13 06:12:19]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2017-01-13 06:12:37]

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-01-13 06:17:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484287939466720,
"parent_rev": "d310c8c6b2f814159adfc056a7a0a4d420604670", "commit_rev":
"9cdca0764fe0ab606d904f8e2bd6eb5117773a97"}

[commit-bot: I haz the power, 2017-01-13 06:18:12]

Description was changed from

==========
Add CHECKs to make sure ThreadableLoader has not been created multiple times

A ThreadableLoader must be completed before it's destructed. We need to forbid
to create ThreadableLoader multiple times in some classes to ensure that.
This CL adds CHECKs to make sure that.

BUG=667254
==========

to

==========
Add CHECKs to make sure ThreadableLoader has not been created multiple times

A ThreadableLoader must be completed before it's destructed. We need to forbid
to create ThreadableLoader multiple times in some classes to ensure that.
This CL adds CHECKs to make sure that.

BUG=667254

Review-Url: https://codereview.chromium.org/2629823002
Cr-Commit-Position: refs/heads/master@{#443506}
Committed:
https://chromium.googlesource.com/chromium/src/+/9cdca0764fe0ab606d904f8e2bd6...
==========

[commit-bot: I haz the power, 2017-01-13 06:18:13]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/9cdca0764fe0ab606d904f8e2bd6...