Make ThreadableLoader class to use references

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 695529aece01bd1ef5b2e2e5074979234a883866..4dbcaa6ee70deafb2ab5aff9699bb9dfcc652daa 100644
--- a/Source/core/page/EventSource.cpp
+++ b/Source/core/page/EventSource.cpp
@@ -125,6 +125,7 @@ void EventSource::connect()
     ASSERT(m_state == CONNECTING);
     ASSERT(!m_requestInFlight);
 
+    ExecutionContext* executionContextLocal = executionContext();

[Inactive, 2014/03/31 17:43:26]

I don't really like the "*Local" naming. I guess you could do:
ExecutionContext* executionContext = this->executionContext();

But why do we even introduce a local variable here?

[maheshkk, 2014/03/31 20:46:49]

Thanks! changed it as per your suggestion.  
Local variable is to limit calling the getter multiple times.

     ResourceRequest request(m_url);
     request.setHTTPMethod("GET");
     request.setHTTPHeaderField("Accept", "text/event-stream");
@@ -132,7 +133,7 @@ void EventSource::connect()
     if (!m_lastEventId.isEmpty())
         request.setHTTPHeaderField("Last-Event-ID", m_lastEventId);
 
-    SecurityOrigin* origin = executionContext()->securityOrigin();
+    SecurityOrigin* origin = executionContextLocal->securityOrigin();
 
     ThreadableLoaderOptions options;
     options.sniffContent = DoNotSniffContent;
@@ -142,9 +143,9 @@ void EventSource::connect()
     options.crossOriginRequestPolicy = UseAccessControl;
     options.dataBufferingPolicy = DoNotBufferData;
     options.securityOrigin = origin;
-    options.contentSecurityPolicyEnforcement = ContentSecurityPolicy::shouldBypassMainWorld(executionContext()) ? DoNotEnforceContentSecurityPolicy : EnforceConnectSrcDirective;
+    options.contentSecurityPolicyEnforcement = ContentSecurityPolicy::shouldBypassMainWorld(executionContextLocal) ? DoNotEnforceContentSecurityPolicy : EnforceConnectSrcDirective;
 
-    m_loader = ThreadableLoader::create(executionContext(), this, request, options);
+    m_loader = ThreadableLoader::create(*executionContextLocal, this, request, options);
 
     if (m_loader)
         m_requestInFlight = true;