HTMLLinkElement should not run script inside ::insertedInto

HTMLLinkElement should not run script inside ::insertedInto

This was done in r165624 but was wrong, you cannot ever synchronously
dispatch the load or error event from inside didFinish since it means
you run can end up running script inside ::insertedInto.

It's not clear from the bug what the issue was, but the timing in the
test that I updated was also wrong since it had <link> elements firing
their load events synchronously while parsing.

There might be a real timing bug in imports, but firing the event
synchronously inside ::didFinish() isn't how to fix it.

BUG=336113, 357435, 357445
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170268

[Hajime Morrita, 2014-03-28 02:00:47]

LGTM.
Could you rather file a bug for the order thing and mark the test fail on it?

[esprehn, 2014-03-28 02:02:28]

On 2014/03/28 02:00:47, morrita1 wrote:
> LGTM.
> Could you rather file a bug for the order thing and mark the test fail on it?

The test passes though, load events should always be async. I updated it for
that.

[esprehn, 2014-03-28 03:49:26]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2014-03-28 03:49:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/esprehn@chromium.org/214513011/1

[commit-bot: I haz the power, 2014-03-28 03:59:55]

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

[commit-bot: I haz the power, 2014-03-28 03:59:56]

Try jobs failed on following builders:
  tryserver.blink on win_blink_compile_dbg

[esprehn, 2014-03-28 04:03:12]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2014-03-28 04:03:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/esprehn@chromium.org/214513011/1

[commit-bot: I haz the power, 2014-03-28 04:57:54]

Change committed as 170268