1. Check that ClassicPendingScript is not accessed after prefinalizer explicitly

Check that ClassicPendingScript is not accessed after prefinalizer explicitly

To ensure upcoming changes to prefinalizers around PendingScript, this CL
asserts that ClassicPendingScript is not touched after prefinalization.

The state transition of ClassicPendingScript and ScriptStreamer is
complicated, and I'd like to check that our change doesn't break the
their assumptions.

BUG=715309
Review-Url: https://codereview.chromium.org/2837413002
Cr-Commit-Position: refs/heads/master@{#467205}
Committed: https://chromium.googlesource.com/chromium/src/+/f72cff34bada6f453c14170273e2c67354209109

[hiroshige, 2017-04-25 23:15:47]

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

[commit-bot: I haz the power, 2017-04-25 23:16:20]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hiroshige, 2017-04-25 23:29:58]

Description was changed from

==========
Check that ClassicPendingScript is not accessed after prefinalizer explicitly

BUG=
==========

to

==========
Check that ClassicPendingScript is not accessed after prefinalizer explicitly

To ensure upcoming changes to prefinalizers around PendingScript, this CL
asserts that ClassicPendingScript is not touched after prefinalization.

The state transition of ClassicPendingScript and ScriptStreamer is
complicated, and I'd like to check that our change doesn't break the
their assumptions.

BUG=715309
==========

[hiroshige, 2017-04-25 23:29:58]

hiroshige@chromium.org changed reviewers:
+ kouhei@chromium.org

[kouhei (in TOK), 2017-04-26 00:21:02]

lgtm

[commit-bot: I haz the power, 2017-04-26 01:14:51]

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

[commit-bot: I haz the power, 2017-04-26 01:14:51]

Dry run: This issue passed the CQ dry run.

[kouhei (in TOK), 2017-04-26 02:07:27]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2017-04-26 02:08:10]

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-26 02:12:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493172447980740, "parent_rev":
"31790e2307fbd3316eb723a97470b913e15557c2", "commit_rev":
"f72cff34bada6f453c14170273e2c67354209109"}

[commit-bot: I haz the power, 2017-04-26 02:12:54]

Description was changed from

==========
Check that ClassicPendingScript is not accessed after prefinalizer explicitly

To ensure upcoming changes to prefinalizers around PendingScript, this CL
asserts that ClassicPendingScript is not touched after prefinalization.

The state transition of ClassicPendingScript and ScriptStreamer is
complicated, and I'd like to check that our change doesn't break the
their assumptions.

BUG=715309
==========

to

==========
Check that ClassicPendingScript is not accessed after prefinalizer explicitly

To ensure upcoming changes to prefinalizers around PendingScript, this CL
asserts that ClassicPendingScript is not touched after prefinalization.

The state transition of ClassicPendingScript and ScriptStreamer is
complicated, and I'd like to check that our change doesn't break the
their assumptions.

BUG=715309

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

[commit-bot: I haz the power, 2017-04-26 02:12:55]

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