Let notifyScriptLoadError() handle already detached ScriptLoaders.

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/282119e0905e8fbb0024e7341e0ae48eb406d2de
Cr-Commit-Position: refs/heads/master@{#372075}

[sof, 2016-01-28 09:20:05]

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

[sof, 2016-01-28 09:20:06]

please take a look.

To ease backporting, I suggest we do this instead..but after having reverted
r371772

[haraken, 2016-01-28 09:23:41]

LGTM

[sof, 2016-01-28 10:00:21]

Description was changed from

==========
Let notifyScriptLoadError() handle already detached ScriptLoaders.

If a ScriptRunner has been disposed of already, allow ScriptLoaders
to notify of their failure without asserting.

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

to

==========
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
==========

[sof, 2016-01-28 10:00:28]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-01-28 10:00:49]

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

[commit-bot: I haz the power, 2016-01-28 11:35:50]

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

[commit-bot: I haz the power, 2016-01-28 11:35:51]

Dry run: This issue passed the CQ dry run.

[sof, 2016-01-28 11:43:45]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-01-28 11:43:45]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1642863002/#ps20001
(title: "rebased up to r372058")

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

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

[commit-bot: I haz the power, 2016-01-28 11:48:48]

Description was changed from

==========
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
==========

to

==========
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
==========

[commit-bot: I haz the power, 2016-01-28 11:48:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-28 11:49:35]

Description was changed from

==========
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
==========

to

==========
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/282119e0905e8fbb0024e7341e0ae48eb406d2de
Cr-Commit-Position: refs/heads/master@{#372075}
==========

[commit-bot: I haz the power, 2016-01-28 11:49:36]

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