Make ThreadableLoader class to use references

Source/core/fileapi/FileReaderLoader.cpp

Index: Source/core/fileapi/FileReaderLoader.cpp
diff --git a/Source/core/fileapi/FileReaderLoader.cpp b/Source/core/fileapi/FileReaderLoader.cpp
index 74e518bcb39cfff9711272e55bbfcaecf3aa1a4d..859b65a802582f462b586c78ba5133556c6ac11e 100644
--- a/Source/core/fileapi/FileReaderLoader.cpp
+++ b/Source/core/fileapi/FileReaderLoader.cpp
@@ -115,19 +115,21 @@ void FileReaderLoader::startInternal(ExecutionContext* executionContext, const S
     options.initiator = FetchInitiatorTypeNames::internal;
 
     if (m_client)
-        m_loader = ThreadableLoader::create(executionContext, this, request, options);
+        m_loader = ThreadableLoader::create(*executionContext, this, request, options);
     else
-        ThreadableLoader::loadResourceSynchronously(executionContext, request, *this, options);
+        ThreadableLoader::loadResourceSynchronously(*executionContext, request, *this, options);
 }
 
 void FileReaderLoader::start(ExecutionContext* executionContext, PassRefPtr<BlobDataHandle> blobData)
 {
+    ASSERT(executionContext);

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

Is it a lot of work to update the callers?

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

I have changed the startInternal to use reference. Most of this class code now
uses reference for executeContext. 

There are 8 different callers to start() and will upload a different patch to
limit the number of files for this patch to review.  

     m_urlForReadingIsStream = false;
     startInternal(executionContext, 0, blobData);
 }
 
 void FileReaderLoader::start(ExecutionContext* executionContext, const Stream& stream, unsigned readSize)
 {
+    ASSERT(executionContext);

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

Ditto.

     if (readSize > 0) {
         m_hasRange = true;
         m_rangeStart = 0;