Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders.

Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders. (patchset #3 id:40001 of https://codereview.chromium.org/1644483002/ )

Reason for revert:
The change here assumes PendingScripts are separately allocated objects, an M50 change. This makes for more difficult backporting. Unnecessarily so.

Hence reverting and will reland a variation ( https://codereview.chromium.org/1642863002/ ) that works for older branches.

Original issue's description:
> Let notifyScriptLoadError() handle already detached ScriptLoaders.
>
> If a ScriptRunner has been disposed of already, allow ScriptLoaders
> to notify of their failure without asserting.
>
> R=haraken
> BUG=570012
>
> Committed: https://crrev.com/e7bf58190483dffac8e78506884170720165b198
> Cr-Commit-Position: refs/heads/master@{#371772}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570012
Committed: https://crrev.com/51bab0b185b4c4ac4ef9ed62241e1b2ef9f8ca0a
Cr-Commit-Position: refs/heads/master@{#372058}

[sof, 2016-01-28 09:44:32]

Created Revert of Let notifyScriptLoadError() handle already detached
ScriptLoaders.

[commit-bot: I haz the power, 2016-01-28 09:44:58]

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

[commit-bot: I haz the power, 2016-01-28 09:45:53]

Description was changed from

==========
Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders.
(patchset #3 id:40001 of https://codereview.chromium.org/1644483002/ )

Reason for revert:
The change here assumes PendingScripts are separately allocated objects, an M50
change. This makes for more difficult backporting. Unnecessarily so.

Hence reverting and will reland a variation (
https://codereview.chromium.org/1642863002/ ) that works for older branches.

Original issue's description:
> Let notifyScriptLoadError() handle already detached ScriptLoaders.
> 
> If a ScriptRunner has been disposed of already, allow ScriptLoaders
> to notify of their failure without asserting.
> 
> R=haraken
> BUG=570012
> 
> Committed: https://crrev.com/e7bf58190483dffac8e78506884170720165b198
> Cr-Commit-Position: refs/heads/master@{#371772}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570012
==========

to

==========
Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders.
(patchset #3 id:40001 of https://codereview.chromium.org/1644483002/ )

Reason for revert:
The change here assumes PendingScripts are separately allocated objects, an M50
change. This makes for more difficult backporting. Unnecessarily so.

Hence reverting and will reland a variation (
https://codereview.chromium.org/1642863002/ ) that works for older branches.

Original issue's description:
> Let notifyScriptLoadError() handle already detached ScriptLoaders.
> 
> If a ScriptRunner has been disposed of already, allow ScriptLoaders
> to notify of their failure without asserting.
> 
> R=haraken
> BUG=570012
> 
> Committed: https://crrev.com/e7bf58190483dffac8e78506884170720165b198
> Cr-Commit-Position: refs/heads/master@{#371772}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570012
==========

[commit-bot: I haz the power, 2016-01-28 09:45:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-28 09:46:45]

Description was changed from

==========
Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders.
(patchset #3 id:40001 of https://codereview.chromium.org/1644483002/ )

Reason for revert:
The change here assumes PendingScripts are separately allocated objects, an M50
change. This makes for more difficult backporting. Unnecessarily so.

Hence reverting and will reland a variation (
https://codereview.chromium.org/1642863002/ ) that works for older branches.

Original issue's description:
> Let notifyScriptLoadError() handle already detached ScriptLoaders.
> 
> If a ScriptRunner has been disposed of already, allow ScriptLoaders
> to notify of their failure without asserting.
> 
> R=haraken
> BUG=570012
> 
> Committed: https://crrev.com/e7bf58190483dffac8e78506884170720165b198
> Cr-Commit-Position: refs/heads/master@{#371772}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570012
==========

to

==========
Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders.
(patchset #3 id:40001 of https://codereview.chromium.org/1644483002/ )

Reason for revert:
The change here assumes PendingScripts are separately allocated objects, an M50
change. This makes for more difficult backporting. Unnecessarily so.

Hence reverting and will reland a variation (
https://codereview.chromium.org/1642863002/ ) that works for older branches.

Original issue's description:
> Let notifyScriptLoadError() handle already detached ScriptLoaders.
>
> If a ScriptRunner has been disposed of already, allow ScriptLoaders
> to notify of their failure without asserting.
>
> R=haraken
> BUG=570012
>
> Committed: https://crrev.com/e7bf58190483dffac8e78506884170720165b198
> Cr-Commit-Position: refs/heads/master@{#371772}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570012

Committed: https://crrev.com/51bab0b185b4c4ac4ef9ed62241e1b2ef9f8ca0a
Cr-Commit-Position: refs/heads/master@{#372058}
==========

[commit-bot: I haz the power, 2016-01-28 09:46:46]

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

[haraken, 2016-01-28 09:47:03]

LGTM