Prevent ThreadableLoaderClient methods from being called after failure notification

Source/core/page/EventSource.cpp

Index: Source/core/page/EventSource.cpp
diff --git a/Source/core/page/EventSource.cpp b/Source/core/page/EventSource.cpp
index 8f893ca29f11ef93ffa3666fd82eee0ac7adba90..6eeffb7a330661ff2b44f819f9a65fc36e7b54af 100644
--- a/Source/core/page/EventSource.cpp
+++ b/Source/core/page/EventSource.cpp
@@ -315,13 +315,10 @@ void EventSource::abortConnectionAttempt()
 {
     ASSERT(m_state == CONNECTING);
 
-    if (m_requestInFlight) {
-        m_loader->cancel();

[hiroshige, 2015/08/03 13:22:19]

What is the reason for omitting cancel() in this CL?

[tyoshino (SeeGerritForStatus), 2015/08/04 08:32:05]

With the main change of this CL, the loader stops working after calling any of
the didFail.* family functions. So, we no longer need to call cancel() in them.

-    } else {
-        m_state = CLOSED;
-    }
+    m_loader = nullptr;
+    m_state = CLOSED;
+    networkRequestEnded();
 
-    ASSERT(m_state == CLOSED);
     dispatchEvent(Event::create(EventTypeNames::error));
 }