Have detached ScriptLoaders detach their PendingScripts.

Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript promptly.

R=
BUG=

Committed: https://crrev.com/cb7d218fb4605744596dbb2b722f3e8934227093
Cr-Commit-Position: refs/heads/master@{#397106}

[sof, 2016-06-01 07:38:27]

Description was changed from

==========
Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript.

R=
BUG=
==========

to

==========
Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript promptly.

R=
BUG=
==========

[sof, 2016-06-01 08:46:08]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, japhet@chromium.org

[sof, 2016-06-01 08:46:08]

please take a look.

spent a long while trying to come up with a layout test that exercises this code
path, but failed. crbug.com/615977 shows that it is not too uncommon for script
loads to end up there, but it involves page navigation GCs & script load timings
afaict.

[haraken, 2016-06-01 11:08:41]

LGTM

[sof, 2016-06-01 11:20:32]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-06-01 11:20:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2028613002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2028613002/1

[commit-bot: I haz the power, 2016-06-01 11:24:08]

Description was changed from

==========
Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript promptly.

R=
BUG=
==========

to

==========
Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript promptly.

R=
BUG=
==========

[commit-bot: I haz the power, 2016-06-01 11:24:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-01 11:25:51]

Description was changed from

==========
Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript promptly.

R=
BUG=
==========

to

==========
Have detached ScriptLoaders detach their PendingScripts.

A ScriptLoader that has become detached from its document will not
execute the script once the script resource has loaded.

If in such a detached state, make sure the ScriptLoader lets go
of its PendingScript promptly.

R=
BUG=

Committed: https://crrev.com/cb7d218fb4605744596dbb2b722f3e8934227093
Cr-Commit-Position: refs/heads/master@{#397106}
==========

[commit-bot: I haz the power, 2016-06-01 11:25:53]

Patchset 1 (id:??) landed as
https://crrev.com/cb7d218fb4605744596dbb2b722f3e8934227093
Cr-Commit-Position: refs/heads/master@{#397106}