Also transfer pending in-order scripts upon element moving to new document

Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=haraken
BUG=570012
Committed: https://crrev.com/3cdc8f849d7afb8404e57103dee2e6aa70c4f8f7
Cr-Commit-Position: refs/heads/master@{#371145}

[sof, 2016-01-23 09:21:46]

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

[sof, 2016-01-23 09:21:47]

please take a look.

(a wider reviewer set so as to potentially give this one a bit more time to
bake, if approved.)

While here, also back out the changes by the speculative r365541 as it is
unnecessary.

[haraken, 2016-01-23 10:27:03]

On 2016/01/23 09:21:47, sof wrote:
> please take a look.
> 
> (a wider reviewer set so as to potentially give this one a bit more time to
> bake, if approved.)
> 
> While here, also back out the changes by the speculative r365541 as it is
> unnecessary.

Thanks for the quick fix to the P0 bug!

LGTM.

[sof, 2016-01-23 11:44:12]

Description was changed from

==========
Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=
BUG=570012
==========

to

==========
Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=haraken
BUG=570012
==========

[sof, 2016-01-23 11:45:30]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-01-23 11:45:37]

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

[commit-bot: I haz the power, 2016-01-23 23:20:38]

Description was changed from

==========
Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=haraken
BUG=570012
==========

to

==========
Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=haraken
BUG=570012
==========

[commit-bot: I haz the power, 2016-01-23 23:20:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-23 23:22:53]

Description was changed from

==========
Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=haraken
BUG=570012
==========

to

==========
Also transfer pending in-order scripts upon element moving to new document

Extend the ScriptLoader script runner reassociation that happens when a
script element moves to a new document to also include pending in-order
scripts.

Having this be restricted to async scripts unnecessarily confuses the
ScriptLoader when it notifies the ScriptRunner.

R=haraken
BUG=570012

Committed: https://crrev.com/3cdc8f849d7afb8404e57103dee2e6aa70c4f8f7
Cr-Commit-Position: refs/heads/master@{#371145}
==========

[commit-bot: I haz the power, 2016-01-23 23:22:53]

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