Don't dispatch two 'load' events for external SVGScriptElement loads

Don't dispatch two 'load' events for external SVGScriptElement loads

If inserting an <svg:script> reference an external source via script,
ScriptLoaded would dispatch 'load' and then SVGScriptElement would
do the same. Drop the latter one.
Since this makes SVGScriptElement no longer use
SVGElement::sendSVGLoadEventIfPossibleAsynchronously(), drop the
svgLoadEventTimer() override and associated timer.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195604

[fs, 2015-05-18 15:17:33]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[fs, 2015-05-19 07:25:40]

https://codereview.chromium.org/1141683003/diff/1/LayoutTests/svg/dom/SVGScri...
File LayoutTests/svg/dom/SVGScriptElement/script-external-no-multiple-load.html
(right):

https://codereview.chromium.org/1141683003/diff/1/LayoutTests/svg/dom/SVGScri...
LayoutTests/svg/dom/SVGScriptElement/script-external-no-multiple-load.html:12:
setTimeout(function() {
Not very fond of this, but did manage to come up with anything better to latch
on to... Suggestions welcome.

[pdr., 2015-05-19 18:18:26]

LGTM. Nice!

https://codereview.chromium.org/1141683003/diff/1/LayoutTests/svg/dom/SVGScri...
File LayoutTests/svg/dom/SVGScriptElement/script-external-no-multiple-load.html
(right):

https://codereview.chromium.org/1141683003/diff/1/LayoutTests/svg/dom/SVGScri...
LayoutTests/svg/dom/SVGScriptElement/script-external-no-multiple-load.html:12:
setTimeout(function() {
On 2015/05/19 at 07:25:40, fs wrote:
> Not very fond of this, but did manage to come up with anything better to latch
on to... Suggestions welcome.

I think this is fine. setTimeout is our hook for waiting for pending tasks.

[fs, 2015-05-20 07:47:32]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-05-20 07:48:05]

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

[commit-bot: I haz the power, 2015-05-20 09:03:51]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195604