Make AsyncFileReader work without ScriptExecutionContext
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Mar 2013 17:42:47 +0000 (17:42 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Mar 2013 17:42:47 +0000 (17:42 +0000)
        https://bugs.webkit.org/show_bug.cgi?id=111376

        Reviewed by Anders Carlsson.

        * WebCore.xcodeproj/project.pbxproj: Added MainThreadTask.h.

        * dom/ScriptExecutionContext.cpp:
        * dom/ScriptExecutionContext.h:
        Removed fileThread(). Creating a thread for every context that acesses files appears
        wasteful and unnecessary.

        * fileapi/AsyncFileStream.h:
        * fileapi/AsyncFileStream.cpp:
        (WebCore::createFileThread): FileThread is now a singleton.
        (WebCore::fileThread): Ditto.
        (WebCore::AsyncFileStream::AsyncFileStream):
        (WebCore::AsyncFileStream::create):
        (WebCore::didStart):
        (WebCore::AsyncFileStream::startOnFileThread):
        (WebCore::AsyncFileStream::stop):
        (WebCore::derefProxyOnMainThread):
        (WebCore::AsyncFileStream::stopOnFileThread):
        (WebCore::didGetSize):
        (WebCore::AsyncFileStream::getSizeOnFileThread):
        (WebCore::didOpen):
        (WebCore::AsyncFileStream::openForReadOnFileThread):
        (WebCore::AsyncFileStream::openForWriteOnFileThread):
        (WebCore::didRead):
        (WebCore::AsyncFileStream::readOnFileThread):
        (WebCore::didWrite):
        (WebCore::AsyncFileStream::writeOnFileThread):
        (WebCore::didTruncate):
        (WebCore::AsyncFileStream::truncateOnFileThread):
        Use callOnMainThread() instead of posting messages through a context. We never used
        other threads, and don't plan to.

        * loader/ResourceLoader.cpp:
        * loader/ResourceLoader.h:
        We no longer need this pseudo-client call to create an AsyncFileStream, now that
        we don't need to get a ScriptExecutionContext somewhere.

        * platform/MainThreadTask.h: Added. This implements callOnMainThread with multiple
        strongly typed arguments. Closely based on CrossThreadTask.h.

        * platform/network/BlobResourceHandle.cpp:
        (WebCore::BlobResourceHandle::BlobResourceHandle): Create AsyncFileStream directly.

        * platform/network/ResourceHandleClient.h: Removed createAsyncFileStream().

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@144770 268f45cc-cd09-0410-ab3c-d52691b4dbfc

19 files changed:
Source/WebCore/ChangeLog
Source/WebCore/GNUmakefile.list.am
Source/WebCore/Target.pri
Source/WebCore/WebCore.vcproj/WebCore.vcproj
Source/WebCore/WebCore.vcxproj/WebCore.vcxproj
Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters
Source/WebCore/WebCore.xcodeproj/project.pbxproj
Source/WebCore/dom/ScriptExecutionContext.cpp
Source/WebCore/dom/ScriptExecutionContext.h
Source/WebCore/fileapi/AsyncFileStream.cpp
Source/WebCore/fileapi/AsyncFileStream.h
Source/WebCore/loader/ResourceLoader.cpp
Source/WebCore/loader/ResourceLoader.h
Source/WebCore/platform/MainThreadTask.h [new file with mode: 0644]
Source/WebCore/platform/network/BlobResourceHandle.cpp
Source/WebCore/platform/network/ResourceHandleClient.h
Source/WebKit2/ChangeLog
Source/WebKit2/NetworkProcess/NetworkResourceLoader.cpp
Source/WebKit2/NetworkProcess/NetworkResourceLoader.h

index 4fac58b..dfec86a 100644 (file)
@@ -1,3 +1,55 @@
+2013-03-04  Alexey Proskuryakov  <ap@apple.com>
+
+        Make AsyncFileReader work without ScriptExecutionContext
+        https://bugs.webkit.org/show_bug.cgi?id=111376
+
+        Reviewed by Anders Carlsson.
+
+        * WebCore.xcodeproj/project.pbxproj: Added MainThreadTask.h.
+
+        * dom/ScriptExecutionContext.cpp:
+        * dom/ScriptExecutionContext.h:
+        Removed fileThread(). Creating a thread for every context that acesses files appears
+        wasteful and unnecessary.
+
+        * fileapi/AsyncFileStream.h:
+        * fileapi/AsyncFileStream.cpp:
+        (WebCore::createFileThread): FileThread is now a singleton.
+        (WebCore::fileThread): Ditto.
+        (WebCore::AsyncFileStream::AsyncFileStream):
+        (WebCore::AsyncFileStream::create):
+        (WebCore::didStart):
+        (WebCore::AsyncFileStream::startOnFileThread):
+        (WebCore::AsyncFileStream::stop):
+        (WebCore::derefProxyOnMainThread):
+        (WebCore::AsyncFileStream::stopOnFileThread):
+        (WebCore::didGetSize):
+        (WebCore::AsyncFileStream::getSizeOnFileThread):
+        (WebCore::didOpen):
+        (WebCore::AsyncFileStream::openForReadOnFileThread):
+        (WebCore::AsyncFileStream::openForWriteOnFileThread):
+        (WebCore::didRead):
+        (WebCore::AsyncFileStream::readOnFileThread):
+        (WebCore::didWrite):
+        (WebCore::AsyncFileStream::writeOnFileThread):
+        (WebCore::didTruncate):
+        (WebCore::AsyncFileStream::truncateOnFileThread):
+        Use callOnMainThread() instead of posting messages through a context. We never used
+        other threads, and don't plan to.
+
+        * loader/ResourceLoader.cpp:
+        * loader/ResourceLoader.h:
+        We no longer need this pseudo-client call to create an AsyncFileStream, now that
+        we don't need to get a ScriptExecutionContext somewhere.
+
+        * platform/MainThreadTask.h: Added. This implements callOnMainThread with multiple
+        strongly typed arguments. Closely based on CrossThreadTask.h.
+
+        * platform/network/BlobResourceHandle.cpp:
+        (WebCore::BlobResourceHandle::BlobResourceHandle): Create AsyncFileStream directly.
+
+        * platform/network/ResourceHandleClient.h: Removed createAsyncFileStream().
+
 2013-03-05  Jessie Berlin  <jberlin@apple.com>
 
         Update the localizable strings file.
index b266b5d..27cd98f 100644 (file)
@@ -5786,6 +5786,7 @@ webcore_platform_sources += \
        Source/WebCore/platform/LocalizedStrings.h \
        Source/WebCore/platform/Logging.cpp \
        Source/WebCore/platform/Logging.h \
+       Source/WebCore/platform/MainThreadTask.h \
        Source/WebCore/platform/MIMETypeRegistry.cpp \
        Source/WebCore/platform/MIMETypeRegistry.h \
        Source/WebCore/platform/MemoryPressureHandler.cpp \
index 0a90142..3b06c04 100644 (file)
@@ -2331,6 +2331,7 @@ HEADERS += \
     platform/Language.h \
     platform/MemoryPressureHandler.h \
     platform/MemoryUsageSupport.h \
+    platform/MainThreadTask.h \
     platform/MIMETypeRegistry.h \
     platform/network/AuthenticationChallengeBase.h \
     platform/network/AuthenticationClient.h \
index 7ed69e8..f3a1bb6 100755 (executable)
                                >
                        </File>
                        <File
+                               RelativePath="..\platform\MainThreadTask.h"
+                               >
+                       </File>
+                       <File
                                RelativePath="..\platform\MIMETypeRegistry.cpp"
                                >
                        </File>
index 684a01c..bf98c1d 100644 (file)
     <ClInclude Include="..\platform\Logging.h" />
     <ClInclude Include="..\platform\MemoryPressureHandler.h" />
     <ClInclude Include="..\platform\MemoryUsageSupport.h" />
+    <ClInclude Include="..\platform\MainThreadTask.h" />
     <ClInclude Include="..\platform\MIMETypeRegistry.h" />
     <ClInclude Include="..\platform\NotImplemented.h" />
     <ClInclude Include="..\platform\Pasteboard.h" />
index 50ca63d..3af2142 100644 (file)
     <ClInclude Include="..\platform\MemoryUsageSupport.h">
       <Filter>platform</Filter>
     </ClInclude>
+    <ClInclude Include="..\platform\MainThreadTask.h">
+      <Filter>platform</Filter>
+    </ClInclude>
     <ClInclude Include="..\platform\MIMETypeRegistry.h">
       <Filter>platform</Filter>
     </ClInclude>
index ca2811c..763c47d 100644 (file)
                E182568F0EF2B02D00933242 /* JSWorkerContext.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E182568D0EF2B02D00933242 /* JSWorkerContext.cpp */; };
                E18256900EF2B02D00933242 /* JSWorkerContext.h in Headers */ = {isa = PBXBuildFile; fileRef = E182568E0EF2B02D00933242 /* JSWorkerContext.h */; };
                E18258AC0EF3CD7000933242 /* JSWorkerContextCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E18258AB0EF3CD7000933242 /* JSWorkerContextCustom.cpp */; };
+               E187056316E54A0D00585E97 /* MainThreadTask.h in Headers */ = {isa = PBXBuildFile; fileRef = E187056216E54A0D00585E97 /* MainThreadTask.h */; };
                E18772F1126E2629003DD586 /* Language.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E18772F0126E2629003DD586 /* Language.cpp */; };
                E1A1470811102B1500EEC0F3 /* ContainerNodeAlgorithms.h in Headers */ = {isa = PBXBuildFile; fileRef = E1A1470711102B1500EEC0F3 /* ContainerNodeAlgorithms.h */; };
                E1A3162D134BC32D007C9A4F /* WebNSAttributedStringExtras.h in Headers */ = {isa = PBXBuildFile; fileRef = E1A3162B134BC32D007C9A4F /* WebNSAttributedStringExtras.h */; settings = {ATTRIBUTES = (Private, ); }; };
                E1CA5CBC0E8CDCAF00E8EF90 /* JSWorkerCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E1CA5CBB0E8CDCAF00E8EF90 /* JSWorkerCustom.cpp */; };
                E1CAA5C60E8BD23600A73ECA /* JSWorker.h in Headers */ = {isa = PBXBuildFile; fileRef = E1CAA5C50E8BD23600A73ECA /* JSWorker.h */; };
                E1CDE92015018ED000862CC5 /* AsyncFileStream.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E1CDE91F15018ED000862CC5 /* AsyncFileStream.cpp */; };
-               E1CDE9221501916900862CC5 /* AsyncFileStream.h in Headers */ = {isa = PBXBuildFile; fileRef = E1CDE9211501916900862CC5 /* AsyncFileStream.h */; };
+               E1CDE9221501916900862CC5 /* AsyncFileStream.h in Headers */ = {isa = PBXBuildFile; fileRef = E1CDE9211501916900862CC5 /* AsyncFileStream.h */; settings = {ATTRIBUTES = (); }; };
                E1E1BF00115FF6FB006F52CA /* WindowsKeyboardCodes.h in Headers */ = {isa = PBXBuildFile; fileRef = E1E1BEFF115FF6FB006F52CA /* WindowsKeyboardCodes.h */; settings = {ATTRIBUTES = (Private, ); }; };
                E1E6EEA40B628DA8005F2F70 /* JSHTMLSelectElement.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E1E6EEA30B628DA8005F2F70 /* JSHTMLSelectElement.cpp */; };
                E1E6EEA80B628DB3005F2F70 /* JSHTMLSelectElement.h in Headers */ = {isa = PBXBuildFile; fileRef = E1E6EEA70B628DB3005F2F70 /* JSHTMLSelectElement.h */; };
                A31C4E4C16E02AA6002F7957 /* OESTextureHalfFloat.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = OESTextureHalfFloat.cpp; path = canvas/OESTextureHalfFloat.cpp; sourceTree = "<group>"; };
                A31C4E4E16E02AB4002F7957 /* OESTextureHalfFloat.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = OESTextureHalfFloat.h; path = canvas/OESTextureHalfFloat.h; sourceTree = "<group>"; };
                A31C4E5016E02AC5002F7957 /* OESTextureHalfFloat.idl */ = {isa = PBXFileReference; lastKnownFileType = text; name = OESTextureHalfFloat.idl; path = canvas/OESTextureHalfFloat.idl; sourceTree = "<group>"; };
-               A31C4E5116E02B08002F7957 /* JSOESTextureHalfFloat.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = JSOESTextureHalfFloat.cpp; path = JSOESTextureHalfFloat.cpp; sourceTree = "<group>"; };
-               A31C4E5316E02B40002F7957 /* JSOESTextureHalfFloat.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = JSOESTextureHalfFloat.h; path = JSOESTextureHalfFloat.h; sourceTree = "<group>"; };
+               A31C4E5116E02B08002F7957 /* JSOESTextureHalfFloat.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSOESTextureHalfFloat.cpp; sourceTree = "<group>"; };
+               A31C4E5316E02B40002F7957 /* JSOESTextureHalfFloat.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSOESTextureHalfFloat.h; sourceTree = "<group>"; };
                A3BB59F11457A40D00AC56FE /* DocumentEventQueue.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = DocumentEventQueue.cpp; sourceTree = "<group>"; };
                A3BB59F21457A40D00AC56FE /* DocumentEventQueue.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = DocumentEventQueue.h; sourceTree = "<group>"; };
                A3E2642E14748991005A8588 /* WorkerEventQueue.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = WorkerEventQueue.cpp; path = workers/WorkerEventQueue.cpp; sourceTree = "<group>"; };
                E182568D0EF2B02D00933242 /* JSWorkerContext.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSWorkerContext.cpp; sourceTree = "<group>"; };
                E182568E0EF2B02D00933242 /* JSWorkerContext.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSWorkerContext.h; sourceTree = "<group>"; };
                E18258AB0EF3CD7000933242 /* JSWorkerContextCustom.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSWorkerContextCustom.cpp; sourceTree = "<group>"; };
+               E187056216E54A0D00585E97 /* MainThreadTask.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MainThreadTask.h; sourceTree = "<group>"; };
                E18772F0126E2629003DD586 /* Language.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = Language.cpp; sourceTree = "<group>"; };
                E1A1470711102B1500EEC0F3 /* ContainerNodeAlgorithms.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ContainerNodeAlgorithms.h; sourceTree = "<group>"; };
                E1A3162B134BC32D007C9A4F /* WebNSAttributedStringExtras.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebNSAttributedStringExtras.h; sourceTree = "<group>"; };
                                935207BD09BD410A00F2038D /* LocalizedStrings.h */,
                                A8239DFE09B3CF8A00B60641 /* Logging.cpp */,
                                A8239DFF09B3CF8A00B60641 /* Logging.h */,
+                               E187056216E54A0D00585E97 /* MainThreadTask.h */,
                                657EDA061385CB97004E0645 /* MemoryPressureHandler.cpp */,
                                657EDA071385CB97004E0645 /* MemoryPressureHandler.h */,
                                D6FDAEF2149C06190037B1E3 /* MemoryUsageSupport.cpp */,
                                A024575216CEAA27000E5671 /* EXTDrawBuffers.h in Headers */,
                                A31C4E4F16E02AB4002F7957 /* OESTextureHalfFloat.h in Headers */,
                                A31C4E5416E02B40002F7957 /* JSOESTextureHalfFloat.h in Headers */,
+                               E187056316E54A0D00585E97 /* MainThreadTask.h in Headers */,
                                7E66E23416D6EB6C00F7E7FF /* WebGLCompressedTextureATC.h in Headers */,
                                7EA30F6A16DFFE7500257D0B /* JSWebGLCompressedTextureATC.h in Headers */,
                                2D8287F716E4A0380086BD00 /* HitTestLocation.h in Headers */,
index d280c62..882fd1a 100644 (file)
@@ -34,7 +34,6 @@
 #include "ErrorEvent.h"
 #include "EventListener.h"
 #include "EventTarget.h"
-#include "FileThread.h"
 #include "MessagePort.h"
 #include "PublicURLManager.h"
 #include "Settings.h"
@@ -127,10 +126,6 @@ ScriptExecutionContext::~ScriptExecutionContext()
         (*iter)->contextDestroyed();
     }
 #if ENABLE(BLOB)
-    if (m_fileThread) {
-        m_fileThread->stop();
-        m_fileThread = 0;
-    }
     if (m_publicURLManager)
         m_publicURLManager->contextDestroyed();
 #endif
@@ -358,16 +353,6 @@ int ScriptExecutionContext::circularSequentialID()
 }
 
 #if ENABLE(BLOB)
-FileThread* ScriptExecutionContext::fileThread()
-{
-    if (!m_fileThread) {
-        m_fileThread = FileThread::create();
-        if (!m_fileThread->start())
-            m_fileThread = 0;
-    }
-    return m_fileThread.get();
-}
-
 PublicURLManager& ScriptExecutionContext::publicURLManager()
 {
     if (!m_publicURLManager)
@@ -420,7 +405,6 @@ void ScriptExecutionContext::reportMemoryUsage(MemoryObjectInfo* memoryObjectInf
     info.addMember(m_pendingExceptions, "pendingExceptions");
 #if ENABLE(BLOB)
     info.addMember(m_publicURLManager, "publicURLManager");
-    info.addMember(m_fileThread, "fileThread");
 #endif
 }
 
index 61e4b58..f634a48 100644 (file)
@@ -62,10 +62,6 @@ class MessagePort;
 class PublicURLManager;
 #endif
 
-#if ENABLE(BLOB)
-class FileThread;
-#endif
-
 class ScriptExecutionContext : public SecurityContext, public Supplementable<ScriptExecutionContext> {
 public:
     ScriptExecutionContext();
@@ -156,11 +152,6 @@ public:
     JSC::JSGlobalData* globalData();
 #endif
 
-#if ENABLE(BLOB)
-    FileThread* fileThread();
-    void stopFileThread();
-#endif
-
     // Interval is in seconds.
     void adjustMinimumTimerInterval(double oldMinimumTimerInterval);
     virtual double minimumTimerInterval() const;
@@ -230,7 +221,6 @@ private:
 
 #if ENABLE(BLOB)
     OwnPtr<PublicURLManager> m_publicURLManager;
-    RefPtr<FileThread> m_fileThread;
 #endif
 
 #if ENABLE(SQL_DATABASE)
index e948a7a..6425267 100644 (file)
 #include "AsyncFileStream.h"
 
 #include "Blob.h"
-#include "CrossThreadTask.h"
 #include "FileStream.h"
 #include "FileStreamClient.h"
 #include "FileThread.h"
 #include "FileThreadTask.h"
-#include "ScriptExecutionContext.h"
+#include "MainThreadTask.h"
+#include <wtf/MainThread.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
 
-inline AsyncFileStream::AsyncFileStream(ScriptExecutionContext* context, FileStreamClient* client)
-    : m_context(context)
-    , m_stream(FileStream::create())
+static PassRefPtr<FileThread> createFileThread()
+{
+    RefPtr<FileThread> thread = FileThread::create();
+    if (!thread->start())
+        return 0;
+    return thread.release();
+}
+
+static FileThread* fileThread()
+{
+    ASSERT(isMainThread());
+    static FileThread* thread = createFileThread().leakRef();
+    return thread;
+}
+
+inline AsyncFileStream::AsyncFileStream(FileStreamClient* client)
+    : m_stream(FileStream::create())
     , m_client(client)
 {
+    ASSERT(isMainThread());
 }
 
-PassRefPtr<AsyncFileStream> AsyncFileStream::create(ScriptExecutionContext* context, FileStreamClient* client)
+PassRefPtr<AsyncFileStream> AsyncFileStream::create(FileStreamClient* client)
 {
-    RefPtr<AsyncFileStream> proxy = adoptRef(new AsyncFileStream(context, client));
+    RefPtr<AsyncFileStream> proxy = adoptRef(new AsyncFileStream(client));
 
     // Hold a reference so that the instance will not get deleted while there are tasks on the file thread.
     // This is balanced by the deref in derefProxyOnContext below.
     proxy->ref();
 
-    proxy->fileThread()->postTask(createFileThreadTask(proxy.get(), &AsyncFileStream::startOnFileThread));
+    fileThread()->postTask(createFileThreadTask(proxy.get(), &AsyncFileStream::startOnFileThread));
 
     return proxy.release();
 }
@@ -70,14 +85,7 @@ AsyncFileStream::~AsyncFileStream()
 {
 }
 
-FileThread* AsyncFileStream::fileThread()
-{
-    ASSERT(m_context->isContextThread());
-    ASSERT(m_context->fileThread());
-    return m_context->fileThread();
-}
-
-static void didStart(ScriptExecutionContext*, AsyncFileStream* proxy)
+static void didStart(AsyncFileStream* proxy)
 {
     if (proxy->client())
         proxy->client()->didStart();
@@ -85,22 +93,23 @@ static void didStart(ScriptExecutionContext*, AsyncFileStream* proxy)
 
 void AsyncFileStream::startOnFileThread()
 {
-    if (!client())
+    // FIXME: It is not correct to check m_client from a secondary thread - stop() could be racing with this check.
+    if (!m_client)
         return;
     m_stream->start();
-    m_context->postTask(createCallbackTask(&didStart, AllowCrossThreadAccess(this)));
+    callOnMainThread(didStart, AllowCrossThreadAccess(this));
 }
 
 void AsyncFileStream::stop()
 {
-    // Clear the client so that we won't be calling callbacks on the client.
+    // Clear the client so that we won't be invoking callbacks on the client.
     setClient(0);
 
     fileThread()->unscheduleTasks(m_stream.get());
     fileThread()->postTask(createFileThreadTask(this, &AsyncFileStream::stopOnFileThread));
 }
 
-static void derefProxyOnContext(ScriptExecutionContext*, AsyncFileStream* proxy)
+static void derefProxyOnMainThread(AsyncFileStream* proxy)
 {
     ASSERT(proxy->hasOneRef());
     proxy->deref();
@@ -109,10 +118,10 @@ static void derefProxyOnContext(ScriptExecutionContext*, AsyncFileStream* proxy)
 void AsyncFileStream::stopOnFileThread()
 {
     m_stream->stop();
-    m_context->postTask(createCallbackTask(&derefProxyOnContext, AllowCrossThreadAccess(this)));
+    callOnMainThread(derefProxyOnMainThread, AllowCrossThreadAccess(this));
 }
 
-static void didGetSize(ScriptExecutionContext*, AsyncFileStream* proxy, long long size)
+static void didGetSize(AsyncFileStream* proxy, long long size)
 {
     if (proxy->client())
         proxy->client()->didGetSize(size);
@@ -126,10 +135,10 @@ void AsyncFileStream::getSize(const String& path, double expectedModificationTim
 void AsyncFileStream::getSizeOnFileThread(const String& path, double expectedModificationTime)
 {
     long long size = m_stream->getSize(path, expectedModificationTime);
-    m_context->postTask(createCallbackTask(&didGetSize, AllowCrossThreadAccess(this), size));
+    callOnMainThread(didGetSize, AllowCrossThreadAccess(this), size);
 }
 
-static void didOpen(ScriptExecutionContext*, AsyncFileStream* proxy, bool success)
+static void didOpen(AsyncFileStream* proxy, bool success)
 {
     if (proxy->client())
         proxy->client()->didOpen(success);
@@ -143,7 +152,7 @@ void AsyncFileStream::openForRead(const String& path, long long offset, long lon
 void AsyncFileStream::openForReadOnFileThread(const String& path, long long offset, long long length)
 {
     bool success = m_stream->openForRead(path, offset, length);
-    m_context->postTask(createCallbackTask(&didOpen, AllowCrossThreadAccess(this), success));
+    callOnMainThread(didOpen, AllowCrossThreadAccess(this), success);
 }
 
 void AsyncFileStream::openForWrite(const String& path)
@@ -156,7 +165,7 @@ void AsyncFileStream::openForWrite(const String& path)
 void AsyncFileStream::openForWriteOnFileThread(const String& path)
 {
     bool success = m_stream->openForWrite(path);
-    m_context->postTask(createCallbackTask(&didOpen, AllowCrossThreadAccess(this), success));
+    callOnMainThread(didOpen, AllowCrossThreadAccess(this), success);
 }
 
 void AsyncFileStream::close()
@@ -169,7 +178,7 @@ void AsyncFileStream::closeOnFileThread()
     m_stream->close();
 }
 
-static void didRead(ScriptExecutionContext*, AsyncFileStream* proxy, int bytesRead)
+static void didRead(AsyncFileStream* proxy, int bytesRead)
 {
     if (proxy->client())
         proxy->client()->didRead(bytesRead);
@@ -185,10 +194,10 @@ void AsyncFileStream::read(char* buffer, int length)
 void AsyncFileStream::readOnFileThread(char* buffer, int length)
 {
     int bytesRead = m_stream->read(buffer, length);
-    m_context->postTask(createCallbackTask(&didRead, AllowCrossThreadAccess(this), bytesRead));
+    callOnMainThread(didRead, AllowCrossThreadAccess(this), bytesRead);
 }
 
-static void didWrite(ScriptExecutionContext*, AsyncFileStream* proxy, int bytesWritten)
+static void didWrite(AsyncFileStream* proxy, int bytesWritten)
 {
     if (proxy->client())
         proxy->client()->didWrite(bytesWritten);
@@ -202,10 +211,10 @@ void AsyncFileStream::write(const KURL& blobURL, long long position, int length)
 void AsyncFileStream::writeOnFileThread(const KURL& blobURL, long long position, int length)
 {
     int bytesWritten = m_stream->write(blobURL, position, length);
-    m_context->postTask(createCallbackTask(&didWrite, AllowCrossThreadAccess(this), bytesWritten));
+    callOnMainThread(didWrite, AllowCrossThreadAccess(this), bytesWritten);
 }
 
-static void didTruncate(ScriptExecutionContext*, AsyncFileStream* proxy, bool success)
+static void didTruncate(AsyncFileStream* proxy, bool success)
 {
     if (proxy->client())
         proxy->client()->didTruncate(success);
@@ -219,7 +228,7 @@ void AsyncFileStream::truncate(long long position)
 void AsyncFileStream::truncateOnFileThread(long long position)
 {
     bool success = m_stream->truncate(position);
-    m_context->postTask(createCallbackTask(&didTruncate, AllowCrossThreadAccess(this), success));
+    callOnMainThread(didTruncate, AllowCrossThreadAccess(this), success);
 }
 
 } // namespace WebCore
index 04be2db..68f23e9 100644 (file)
@@ -42,13 +42,11 @@ namespace WebCore {
 
 class FileStreamClient;
 class FileStream;
-class FileThread;
 class KURL;
-class ScriptExecutionContext;
 
 class AsyncFileStream : public RefCounted<AsyncFileStream> {
 public:
-    static PassRefPtr<AsyncFileStream> create(ScriptExecutionContext*, FileStreamClient*);
+    static PassRefPtr<AsyncFileStream> create(FileStreamClient*);
     ~AsyncFileStream();
 
     void getSize(const String& path, double expectedModificationTime);
@@ -67,9 +65,7 @@ public:
     void setClient(FileStreamClient* client) { m_client = client; }
 
 private:
-    AsyncFileStream(ScriptExecutionContext*, FileStreamClient*);
-
-    FileThread* fileThread();
+    AsyncFileStream(FileStreamClient*);
 
     // Called on File thread.
     void startOnFileThread();
@@ -82,7 +78,6 @@ private:
     void writeOnFileThread(const KURL& blobURL, long long position, int length);
     void truncateOnFileThread(long long position);
 
-    RefPtr<ScriptExecutionContext> m_context;
     RefPtr<FileStream> m_stream;
 
     FileStreamClient* m_client;
index be7e3ce..10b4a79 100644 (file)
@@ -545,14 +545,6 @@ void ResourceLoader::receivedCancellation(const AuthenticationChallenge&)
     cancel();
 }
 
-#if ENABLE(BLOB)
-AsyncFileStream* ResourceLoader::createAsyncFileStream(FileStreamClient* client)
-{
-    // It is OK to simply return a pointer since AsyncFileStream::create adds an extra ref.
-    return AsyncFileStream::create(m_frame->document()->scriptExecutionContext(), client).get();
-}
-#endif
-
 void ResourceLoader::reportMemoryUsage(MemoryObjectInfo* memoryObjectInfo) const
 {
     MemoryClassInfo info(memoryObjectInfo, this, WebCoreMemoryTypes::Loader);
index 18dba7f..a89fac0 100644 (file)
@@ -134,9 +134,6 @@ public:
     virtual void didDownloadData(ResourceHandle*, int);
     virtual void didDownloadData(int);
 #endif
-#if ENABLE(BLOB)
-    virtual AsyncFileStream* createAsyncFileStream(FileStreamClient*);
-#endif
 
     const KURL& url() const { return m_request.url(); } 
     ResourceHandle* handle() const { return m_handle.get(); }
diff --git a/Source/WebCore/platform/MainThreadTask.h b/Source/WebCore/platform/MainThreadTask.h
new file mode 100644 (file)
index 0000000..819898e
--- /dev/null
@@ -0,0 +1,494 @@
+/*
+ * Copyright (C) 2009-2010 Google Inc. All rights reserved.
+ * Copyright (C) 2013 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef MainThreadTask_h
+#define MainThreadTask_h
+
+#include "CrossThreadCopier.h"
+#include <memory>
+#include <wtf/MainThread.h>
+#include <wtf/PassOwnPtr.h>
+#include <wtf/PassRefPtr.h>
+#include <wtf/TypeTraits.h>
+
+namespace WebCore {
+
+// Traits for the MainThreadTask.
+template<typename T> struct MainThreadTaskTraits {
+    typedef const T& ParamType;
+};
+
+template<typename T> struct MainThreadTaskTraits<T*> {
+    typedef T* ParamType;
+};
+
+template<typename T> struct MainThreadTaskTraits<PassRefPtr<T> > {
+    typedef PassRefPtr<T> ParamType;
+};
+
+template<typename T> struct MainThreadTaskTraits<PassOwnPtr<T> > {
+    typedef PassOwnPtr<T> ParamType;
+};
+
+class MainThreadTaskBase {
+WTF_MAKE_NONCOPYABLE(MainThreadTaskBase);
+WTF_MAKE_FAST_ALLOCATED;
+public:
+    MainThreadTaskBase() { }
+    virtual void performTask() = 0;
+    virtual ~MainThreadTaskBase() { }
+};
+
+template<typename P1, typename MP1>
+class MainThreadTask1 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1);
+    typedef MainThreadTask1<P1, MP1> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1));
+    }
+
+private:
+    MainThreadTask1(Method method, Param1 parameter1)
+        : m_method(method)
+        , m_parameter1(parameter1)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2>
+class MainThreadTask2 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2);
+    typedef MainThreadTask2<P1, MP1, P2, MP2> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2));
+    }
+
+private:
+    MainThreadTask2(Method method, Param1 parameter1, Param2 parameter2)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3>
+class MainThreadTask3 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2, MP3);
+    typedef MainThreadTask3<P1, MP1, P2, MP2, P3, MP3> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+    typedef typename MainThreadTaskTraits<P3>::ParamType Param3;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2, parameter3));
+    }
+
+private:
+    MainThreadTask3(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+        , m_parameter3(parameter3)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2, m_parameter3);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+    P3 m_parameter3;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4>
+class MainThreadTask4 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2, MP3, MP4);
+    typedef MainThreadTask4<P1, MP1, P2, MP2, P3, MP3, P4, MP4> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+    typedef typename MainThreadTaskTraits<P3>::ParamType Param3;
+    typedef typename MainThreadTaskTraits<P4>::ParamType Param4;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2, parameter3, parameter4));
+    }
+
+private:
+    MainThreadTask4(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+        , m_parameter3(parameter3)
+        , m_parameter4(parameter4)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2, m_parameter3, m_parameter4);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+    P3 m_parameter3;
+    P4 m_parameter4;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5>
+class MainThreadTask5 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2, MP3, MP4, MP5);
+    typedef MainThreadTask5<P1, MP1, P2, MP2, P3, MP3, P4, MP4, P5, MP5> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+    typedef typename MainThreadTaskTraits<P3>::ParamType Param3;
+    typedef typename MainThreadTaskTraits<P4>::ParamType Param4;
+    typedef typename MainThreadTaskTraits<P5>::ParamType Param5;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2, parameter3, parameter4, parameter5));
+    }
+
+private:
+    MainThreadTask5(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+        , m_parameter3(parameter3)
+        , m_parameter4(parameter4)
+        , m_parameter5(parameter5)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2, m_parameter3, m_parameter4, m_parameter5);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+    P3 m_parameter3;
+    P4 m_parameter4;
+    P5 m_parameter5;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5, typename P6, typename MP6>
+class MainThreadTask6 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2, MP3, MP4, MP5, MP6);
+    typedef MainThreadTask6<P1, MP1, P2, MP2, P3, MP3, P4, MP4, P5, MP5, P6, MP6> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+    typedef typename MainThreadTaskTraits<P3>::ParamType Param3;
+    typedef typename MainThreadTaskTraits<P4>::ParamType Param4;
+    typedef typename MainThreadTaskTraits<P5>::ParamType Param5;
+    typedef typename MainThreadTaskTraits<P6>::ParamType Param6;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5, Param6 parameter6)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2, parameter3, parameter4, parameter5, parameter6));
+    }
+
+private:
+    MainThreadTask6(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5, Param6 parameter6)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+        , m_parameter3(parameter3)
+        , m_parameter4(parameter4)
+        , m_parameter5(parameter5)
+        , m_parameter6(parameter6)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2, m_parameter3, m_parameter4, m_parameter5, m_parameter6);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+    P3 m_parameter3;
+    P4 m_parameter4;
+    P5 m_parameter5;
+    P6 m_parameter6;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5, typename P6, typename MP6, typename P7, typename MP7>
+class MainThreadTask7 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2, MP3, MP4, MP5, MP6, MP7);
+    typedef MainThreadTask7<P1, MP1, P2, MP2, P3, MP3, P4, MP4, P5, MP5, P6, MP6, P7, MP7> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+    typedef typename MainThreadTaskTraits<P3>::ParamType Param3;
+    typedef typename MainThreadTaskTraits<P4>::ParamType Param4;
+    typedef typename MainThreadTaskTraits<P5>::ParamType Param5;
+    typedef typename MainThreadTaskTraits<P6>::ParamType Param6;
+    typedef typename MainThreadTaskTraits<P7>::ParamType Param7;
+
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5, Param6 parameter6, Param7 parameter7)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2, parameter3, parameter4, parameter5, parameter6, parameter7));
+    }
+
+private:
+    MainThreadTask7(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5, Param6 parameter6, Param7 parameter7)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+        , m_parameter3(parameter3)
+        , m_parameter4(parameter4)
+        , m_parameter5(parameter5)
+        , m_parameter6(parameter6)
+        , m_parameter7(parameter7)
+    {
+    }
+
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2, m_parameter3, m_parameter4, m_parameter5, m_parameter6, m_parameter7);
+    }
+
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+    P3 m_parameter3;
+    P4 m_parameter4;
+    P5 m_parameter5;
+    P6 m_parameter6;
+    P7 m_parameter7;
+};
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5, typename P6, typename MP6, typename P7, typename MP7, typename P8, typename MP8>
+class MainThreadTask8 : public MainThreadTaskBase {
+public:
+    typedef void (*Method)(MP1, MP2, MP3, MP4, MP5, MP6, MP7, MP8);
+    typedef MainThreadTask8<P1, MP1, P2, MP2, P3, MP3, P4, MP4, P5, MP5, P6, MP6, P7, MP7, P8, MP8> MainThreadTask;
+    typedef typename MainThreadTaskTraits<P1>::ParamType Param1;
+    typedef typename MainThreadTaskTraits<P2>::ParamType Param2;
+    typedef typename MainThreadTaskTraits<P3>::ParamType Param3;
+    typedef typename MainThreadTaskTraits<P4>::ParamType Param4;
+    typedef typename MainThreadTaskTraits<P5>::ParamType Param5;
+    typedef typename MainThreadTaskTraits<P6>::ParamType Param6;
+    typedef typename MainThreadTaskTraits<P7>::ParamType Param7;
+    typedef typename MainThreadTaskTraits<P8>::ParamType Param8;
+    
+    static PassOwnPtr<MainThreadTask> create(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5, Param6 parameter6, Param7 parameter7, Param8 parameter8)
+    {
+        return adoptPtr(new MainThreadTask(method, parameter1, parameter2, parameter3, parameter4, parameter5, parameter6, parameter7, parameter8));
+    }
+    
+private:
+    MainThreadTask8(Method method, Param1 parameter1, Param2 parameter2, Param3 parameter3, Param4 parameter4, Param5 parameter5, Param6 parameter6, Param7 parameter7, Param8 parameter8)
+        : m_method(method)
+        , m_parameter1(parameter1)
+        , m_parameter2(parameter2)
+        , m_parameter3(parameter3)
+        , m_parameter4(parameter4)
+        , m_parameter5(parameter5)
+        , m_parameter6(parameter6)
+        , m_parameter7(parameter7)
+        , m_parameter8(parameter8)
+    {
+    }
+    
+    virtual void performTask() OVERRIDE
+    {
+        (*m_method)(m_parameter1, m_parameter2, m_parameter3, m_parameter4, m_parameter5, m_parameter6, m_parameter7, m_parameter8);
+    }
+    
+private:
+    Method m_method;
+    P1 m_parameter1;
+    P2 m_parameter2;
+    P3 m_parameter3;
+    P4 m_parameter4;
+    P5 m_parameter5;
+    P6 m_parameter6;
+    P7 m_parameter7;
+    P8 m_parameter8;
+};
+
+static void executeMainThreadTask(void* context)
+{
+    OwnPtr<MainThreadTaskBase> task = adoptPtr(static_cast<MainThreadTaskBase*>(context));
+    task->performTask();
+}
+
+template<typename P1, typename MP1>
+void callOnMainThread(
+    void (*method)(MP1),
+    const P1& parameter1)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask1<typename CrossThreadCopier<P1>::Type, MP1>::create(
+        method,
+        CrossThreadCopier<P1>::copy(parameter1)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2>
+void callOnMainThread(
+    void (*method)(MP1, MP2),
+    const P1& parameter1, const P2& parameter2)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask2<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2>::create(
+        method,
+        CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3>
+void callOnMainThread(
+    void (*method)(MP1, MP2, MP3),
+    const P1& parameter1, const P2& parameter2, const P3& parameter3)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask3<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2, typename CrossThreadCopier<P3>::Type, MP3>::create(
+        method,
+        CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2),
+        CrossThreadCopier<P3>::copy(parameter3)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4>
+void callOnMainThread(
+    void (*method)(MP1, MP2, MP3, MP4),
+    const P1& parameter1, const P2& parameter2, const P3& parameter3, const P4& parameter4)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask4<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2, typename CrossThreadCopier<P3>::Type, MP3,
+        typename CrossThreadCopier<P4>::Type, MP4>::create(
+            method,
+            CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2),
+            CrossThreadCopier<P3>::copy(parameter3), CrossThreadCopier<P4>::copy(parameter4)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5>
+void callOnMainThread(
+    void (*method)(MP1, MP2, MP3, MP4, MP5),
+    const P1& parameter1, const P2& parameter2, const P3& parameter3, const P4& parameter4, const P5& parameter5)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask5<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2, typename CrossThreadCopier<P3>::Type, MP3,
+        typename CrossThreadCopier<P4>::Type, MP4, typename CrossThreadCopier<P5>::Type, MP5>::create(
+            method,
+            CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2),
+            CrossThreadCopier<P3>::copy(parameter3), CrossThreadCopier<P4>::copy(parameter4),
+            CrossThreadCopier<P5>::copy(parameter5)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5, typename P6, typename MP6>
+void callOnMainThread(
+    void (*method)(MP1, MP2, MP3, MP4, MP5, MP6),
+    const P1& parameter1, const P2& parameter2, const P3& parameter3, const P4& parameter4, const P5& parameter5, const P6& parameter6)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask6<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2, typename CrossThreadCopier<P3>::Type, MP3,
+        typename CrossThreadCopier<P4>::Type, MP4, typename CrossThreadCopier<P5>::Type, MP5, typename CrossThreadCopier<P6>::Type, MP6>::create(
+            method,
+            CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2),
+            CrossThreadCopier<P3>::copy(parameter3), CrossThreadCopier<P4>::copy(parameter4),
+            CrossThreadCopier<P5>::copy(parameter5), CrossThreadCopier<P6>::copy(parameter6)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5, typename P6, typename MP6, typename P7, typename MP7>
+void callOnMainThread(
+    void (*method)(MP1, MP2, MP3, MP4, MP5, MP6, MP7),
+    const P1& parameter1, const P2& parameter2, const P3& parameter3, const P4& parameter4, const P5& parameter5, const P6& parameter6, const P7& parameter7)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask7<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2, typename CrossThreadCopier<P3>::Type, MP3,
+        typename CrossThreadCopier<P4>::Type, MP4, typename CrossThreadCopier<P5>::Type, MP5, typename CrossThreadCopier<P6>::Type, MP6,
+        typename CrossThreadCopier<P7>::Type, MP7>::create(
+            method,
+            CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2),
+            CrossThreadCopier<P3>::copy(parameter3), CrossThreadCopier<P4>::copy(parameter4),
+            CrossThreadCopier<P5>::copy(parameter5), CrossThreadCopier<P6>::copy(parameter6),
+            CrossThreadCopier<P7>::copy(parameter7)).leakPtr());
+}
+
+template<typename P1, typename MP1, typename P2, typename MP2, typename P3, typename MP3, typename P4, typename MP4, typename P5, typename MP5, typename P6, typename MP6, typename P7, typename MP7, typename P8, typename MP8>
+void callOnMainThread(
+    void (*method)(MP1, MP2, MP3, MP4, MP5, MP6, MP7, MP8),
+    const P1& parameter1, const P2& parameter2, const P3& parameter3, const P4& parameter4, const P5& parameter5, const P6& parameter6, const P7& parameter7, const P8& parameter8)
+{
+    WTF::callOnMainThread(executeMainThreadTask, MainThreadTask8<typename CrossThreadCopier<P1>::Type, MP1, typename CrossThreadCopier<P2>::Type, MP2, typename CrossThreadCopier<P3>::Type, MP3,
+    typename CrossThreadCopier<P4>::Type, MP4, typename CrossThreadCopier<P5>::Type, MP5, typename CrossThreadCopier<P6>::Type, MP6,
+    typename CrossThreadCopier<P7>::Type, MP7, typename CrossThreadCopier<P8>::Type, MP8>::create(
+                                                       method,
+                                                       CrossThreadCopier<P1>::copy(parameter1), CrossThreadCopier<P2>::copy(parameter2),
+                                                       CrossThreadCopier<P3>::copy(parameter3), CrossThreadCopier<P4>::copy(parameter4),
+                                                       CrossThreadCopier<P5>::copy(parameter5), CrossThreadCopier<P6>::copy(parameter6),
+                                                       CrossThreadCopier<P7>::copy(parameter7), CrossThreadCopier<P8>::copy(parameter8)).leakPtr());
+}
+
+} // namespace WebCore
+
+
+#endif // MainThreadTask_h
index c41ee55..4a18825 100644 (file)
@@ -168,9 +168,9 @@ BlobResourceHandle::BlobResourceHandle(PassRefPtr<BlobStorageData> blobData, con
     , m_sizeItemCount(0)
     , m_readItemCount(0)
     , m_fileOpened(false)
-{    
+{
     if (m_async)
-        m_asyncStream = client->createAsyncFileStream(this);
+        m_asyncStream = AsyncFileStream::create(this);
     else
         m_stream = FileStream::create();
 }
index 90e631e..cd08803 100644 (file)
@@ -132,9 +132,6 @@ namespace WebCore {
 #if PLATFORM(CHROMIUM)
         virtual void didDownloadData(ResourceHandle*, int /*dataLength*/) { }
 #endif
-#if ENABLE(BLOB)
-        virtual AsyncFileStream* createAsyncFileStream(FileStreamClient*) { return 0; }
-#endif
 
 #if USE(SOUP)
 private:
index 5d93af8..bacec33 100644 (file)
@@ -1,3 +1,14 @@
+2013-03-04  Alexey Proskuryakov  <ap@apple.com>
+
+        Make AsyncFileReader work without ScriptExecutionContext
+        https://bugs.webkit.org/show_bug.cgi?id=111376
+
+        Reviewed by Anders Carlsson.
+
+        * NetworkProcess/NetworkResourceLoader.cpp:
+        * NetworkProcess/NetworkResourceLoader.h:
+        Removed a dummy implementation of createAsyncFileStream, which is no longer needed.
+
 2013-03-05  Alexey Proskuryakov  <ap@apple.com>
 
         <rdar://problem/13345116> REGRESSION: Child processes get access to user cache
index bf24990..43b4b91 100644 (file)
@@ -291,14 +291,6 @@ void NetworkResourceLoader::willStopBufferingData(WebCore::ResourceHandle*, cons
 }
 #endif // PLATFORM(MAC)
 
-#if ENABLE(BLOB)
-WebCore::AsyncFileStream* NetworkResourceLoader::createAsyncFileStream(WebCore::FileStreamClient*)
-{
-    notImplemented();
-    return 0;
-}
-#endif
-
 } // namespace WebKit
 
 #endif // ENABLE(NETWORK_PROCESS)
index dfacb6c..35674f9 100644 (file)
@@ -91,10 +91,6 @@ public:
     virtual void willStopBufferingData(WebCore::ResourceHandle*, const char*, int) OVERRIDE;
 #endif // PLATFORM(MAC)
 
-#if ENABLE(BLOB)
-    virtual WebCore::AsyncFileStream* createAsyncFileStream(WebCore::FileStreamClient*) OVERRIDE;
-#endif
-
 private:
     NetworkResourceLoader(const NetworkResourceLoadParameters&, NetworkConnectionToWebProcess*);