DocumentThreadableLoader should clear its Resource before destruction

DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

BUG=587663
Committed: https://crrev.com/7b53267a2d52ee4449732076860fbfe344852f6e
Cr-Commit-Position: refs/heads/master@{#409714}

[yhirano, 2016-07-28 01:52:55]

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

[commit-bot: I haz the power, 2016-07-28 01:53:08]

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

[yhirano, 2016-07-28 01:58:00]

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

[commit-bot: I haz the power, 2016-07-28 01:58:21]

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, 2016-07-28 03:29:49]

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

[commit-bot: I haz the power, 2016-07-28 03:29:50]

Dry run: This issue passed the CQ dry run.

[yhirano, 2016-07-28 03:41:50]

Description was changed from

==========
DocumentThreadableLoader refactoring: clear resource in didFinishLoading

BUG=587663
==========

to

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

This CL is a refactoring and does not change the behavior.

BUG=587663
==========

[yhirano, 2016-07-28 03:41:50]

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

[yhirano, 2016-07-28 03:42:05]

Description was changed from

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

This CL is a refactoring and does not change the behavior.

BUG=587663
==========

to

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

BUG=587663
==========

[yhirano, 2016-07-28 03:45:49]

Note: A DocumentThreadableLoader can assume the loading is completed when it's
destroyed: see https://codereview.chromium.org/2146403004/.

[tyoshino (SeeGerritForStatus), 2016-07-28 07:50:59]

lgtm

[yhirano, 2016-08-01 07:58:33]

Nate: ping

[Nate Chapin, 2016-08-03 18:15:35]

lgtm

[yhirano, 2016-08-04 00:51:41]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-08-04 00:52:14]

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-08-04 03:26:26]

Description was changed from

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

BUG=587663
==========

to

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

BUG=587663
==========

[commit-bot: I haz the power, 2016-08-04 03:26:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-04 03:29:59]

Description was changed from

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

BUG=587663
==========

to

==========
DocumentThreadableLoader should clear its Resource before destruction

This CL makes sure that DocumentThreadableLoader's associated Resource is
always cleared before destruction.

BUG=587663

Committed: https://crrev.com/7b53267a2d52ee4449732076860fbfe344852f6e
Cr-Commit-Position: refs/heads/master@{#409714}
==========

[commit-bot: I haz the power, 2016-08-04 03:30:00]

Patchset 2 (id:??) landed as
https://crrev.com/7b53267a2d52ee4449732076860fbfe344852f6e
Cr-Commit-Position: refs/heads/master@{#409714}