Assertion hit / crash in EventSource::abortConnectionAttempt()

Assertion hit / crash in EventSource::abortConnectionAttempt()

Fix a possible assertion hit in EventSource::abortConnectionAttempt() when attempting a
cross-origin request towards a non-HTTP URL, an early verification of the protocol
scheme will cause didFailAccessControlCheck to be called synchronously before the loader
has even finished being created. This special case was not handled properly, since we
tried to cancel a non-existing loader, which resulted in a crash.

In addition to checking whether a request is in flight before trying to cancel it, this
change also schedules the initial creation of the loader to happen asynchronously when
when an EventSource is constructed, so that a script can register for the error event
before it is dispatched (as a result of passing a non-HTTP URL to the constructor).

This patch is based on WebKit r149098 by Per-Erik Brodin <per-erik.brodin@ericsson.com>;.

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

[Ken Russell (switch to Gerrit), 2013-11-05 19:30:58]

The patch looks reasonable to me but I'm really not the best person to review
it. Please ask abarth and japhet.

[abarth-chromium, 2013-11-06 18:02:04]

lgtm

[commit-bot: I haz the power, 2013-11-06 18:02:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/60493003/1

[commit-bot: I haz the power, 2013-11-06 19:20:29]

Change committed as 161464