Diff
Modified: trunk/Source/WebCore/ChangeLog (197550 => 197551)
--- trunk/Source/WebCore/ChangeLog 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/ChangeLog 2016-03-04 08:27:36 UTC (rev 197551)
@@ -1,3 +1,23 @@
+2016-03-04 Youenn Fablet <[email protected]>
+
+ Remove PassRefPtr from ThreadableLoader and relatives
+ https://bugs.webkit.org/show_bug.cgi?id=154966
+
+ Reviewed by Darin Adler.
+
+ Covered by existing tests.
+
+ * loader/DocumentThreadableLoader.cpp:
+ (WebCore::DocumentThreadableLoader::create):
+ * loader/DocumentThreadableLoader.h:
+ * loader/ThreadableLoader.cpp:
+ (WebCore::ThreadableLoader::create):
+ * loader/ThreadableLoader.h:
+ * loader/WorkerThreadableLoader.cpp:
+ (WebCore::WorkerThreadableLoader::WorkerThreadableLoader):
+ (WebCore::WorkerThreadableLoader::MainThreadBridge::MainThreadBridge):
+ * loader/WorkerThreadableLoader.h:
+
2016-03-03 Ryosuke Niwa <[email protected]>
Let XCode have its own way and also sort the file.
Modified: trunk/Source/WebCore/loader/DocumentThreadableLoader.cpp (197550 => 197551)
--- trunk/Source/WebCore/loader/DocumentThreadableLoader.cpp 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/loader/DocumentThreadableLoader.cpp 2016-03-04 08:27:36 UTC (rev 197551)
@@ -66,17 +66,17 @@
loadResourceSynchronously(document, request, client, options, nullptr);
}
-PassRefPtr<DocumentThreadableLoader> DocumentThreadableLoader::create(Document& document, ThreadableLoaderClient& client, const ResourceRequest& request, const ThreadableLoaderOptions& options, std::unique_ptr<ContentSecurityPolicy>&& contentSecurityPolicy)
+RefPtr<DocumentThreadableLoader> DocumentThreadableLoader::create(Document& document, ThreadableLoaderClient& client, const ResourceRequest& request, const ThreadableLoaderOptions& options, std::unique_ptr<ContentSecurityPolicy>&& contentSecurityPolicy)
{
RefPtr<DocumentThreadableLoader> loader = adoptRef(new DocumentThreadableLoader(document, client, LoadAsynchronously, request, options, WTFMove(contentSecurityPolicy)));
if (!loader->m_resource)
loader = nullptr;
- return loader.release();
+ return loader;
}
-PassRefPtr<DocumentThreadableLoader> DocumentThreadableLoader::create(Document& document, ThreadableLoaderClient& client, const ResourceRequest& request, const ThreadableLoaderOptions& options)
+RefPtr<DocumentThreadableLoader> DocumentThreadableLoader::create(Document& document, ThreadableLoaderClient& client, const ResourceRequest& request, const ThreadableLoaderOptions& options)
{
- return DocumentThreadableLoader::create(document, client, request, options, nullptr);
+ return create(document, client, request, options, nullptr);
}
DocumentThreadableLoader::DocumentThreadableLoader(Document& document, ThreadableLoaderClient& client, BlockingBehavior blockingBehavior, const ResourceRequest& request, const ThreadableLoaderOptions& options, std::unique_ptr<ContentSecurityPolicy>&& contentSecurityPolicy)
Modified: trunk/Source/WebCore/loader/DocumentThreadableLoader.h (197550 => 197551)
--- trunk/Source/WebCore/loader/DocumentThreadableLoader.h 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/loader/DocumentThreadableLoader.h 2016-03-04 08:27:36 UTC (rev 197551)
@@ -34,8 +34,6 @@
#include "CachedRawResourceClient.h"
#include "CachedResourceHandle.h"
#include "ThreadableLoader.h"
-#include <wtf/PassRefPtr.h>
-#include <wtf/RefCounted.h>
namespace WebCore {
class CachedRawResource;
@@ -52,8 +50,8 @@
static void loadResourceSynchronously(Document&, const ResourceRequest&, ThreadableLoaderClient&, const ThreadableLoaderOptions&, std::unique_ptr<ContentSecurityPolicy>&&);
static void loadResourceSynchronously(Document&, const ResourceRequest&, ThreadableLoaderClient&, const ThreadableLoaderOptions&);
- static PassRefPtr<DocumentThreadableLoader> create(Document&, ThreadableLoaderClient&, const ResourceRequest&, const ThreadableLoaderOptions&, std::unique_ptr<ContentSecurityPolicy>&&);
- static PassRefPtr<DocumentThreadableLoader> create(Document&, ThreadableLoaderClient&, const ResourceRequest&, const ThreadableLoaderOptions&);
+ static RefPtr<DocumentThreadableLoader> create(Document&, ThreadableLoaderClient&, const ResourceRequest&, const ThreadableLoaderOptions&, std::unique_ptr<ContentSecurityPolicy>&&);
+ static RefPtr<DocumentThreadableLoader> create(Document&, ThreadableLoaderClient&, const ResourceRequest&, const ThreadableLoaderOptions&);
virtual ~DocumentThreadableLoader();
Modified: trunk/Source/WebCore/loader/ThreadableLoader.cpp (197550 => 197551)
--- trunk/Source/WebCore/loader/ThreadableLoader.cpp 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/loader/ThreadableLoader.cpp 2016-03-04 08:27:36 UTC (rev 197551)
@@ -69,7 +69,7 @@
return std::make_unique<ThreadableLoaderOptions>(*this, preflightPolicy, crossOriginRequestPolicy, contentSecurityPolicyEnforcement, WTFMove(securityOriginCopy), initiator.isolatedCopy());
}
-PassRefPtr<ThreadableLoader> ThreadableLoader::create(ScriptExecutionContext* context, ThreadableLoaderClient* client, const ResourceRequest& request, const ThreadableLoaderOptions& options)
+RefPtr<ThreadableLoader> ThreadableLoader::create(ScriptExecutionContext* context, ThreadableLoaderClient* client, const ResourceRequest& request, const ThreadableLoaderOptions& options)
{
ASSERT(client);
ASSERT(context);
Modified: trunk/Source/WebCore/loader/ThreadableLoader.h (197550 => 197551)
--- trunk/Source/WebCore/loader/ThreadableLoader.h 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/loader/ThreadableLoader.h 2016-03-04 08:27:36 UTC (rev 197551)
@@ -33,7 +33,6 @@
#include "ResourceLoaderOptions.h"
#include <wtf/Noncopyable.h>
-#include <wtf/PassRefPtr.h>
#include <wtf/RefPtr.h>
#include <wtf/Vector.h>
#include <wtf/text/AtomicString.h>
@@ -86,7 +85,7 @@
WTF_MAKE_NONCOPYABLE(ThreadableLoader);
public:
static void loadResourceSynchronously(ScriptExecutionContext*, const ResourceRequest&, ThreadableLoaderClient&, const ThreadableLoaderOptions&);
- static PassRefPtr<ThreadableLoader> create(ScriptExecutionContext*, ThreadableLoaderClient*, const ResourceRequest&, const ThreadableLoaderOptions&);
+ static RefPtr<ThreadableLoader> create(ScriptExecutionContext*, ThreadableLoaderClient*, const ResourceRequest&, const ThreadableLoaderOptions&);
virtual void cancel() = 0;
void ref() { refThreadableLoader(); }
Modified: trunk/Source/WebCore/loader/WorkerThreadableLoader.cpp (197550 => 197551)
--- trunk/Source/WebCore/loader/WorkerThreadableLoader.cpp 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/loader/WorkerThreadableLoader.cpp 2016-03-04 08:27:36 UTC (rev 197551)
@@ -52,7 +52,7 @@
WorkerThreadableLoader::WorkerThreadableLoader(WorkerGlobalScope* workerGlobalScope, ThreadableLoaderClient* client, const String& taskMode, const ResourceRequest& request, const ThreadableLoaderOptions& options)
: m_workerGlobalScope(workerGlobalScope)
, m_workerClientWrapper(ThreadableLoaderClientWrapper::create(client))
- , m_bridge(*(new MainThreadBridge(m_workerClientWrapper, workerGlobalScope->thread().workerLoaderProxy(), taskMode, request, options, workerGlobalScope->url().strippedForUseAsReferrer(), workerGlobalScope->securityOrigin(), workerGlobalScope->contentSecurityPolicy())))
+ , m_bridge(*new MainThreadBridge(*m_workerClientWrapper, workerGlobalScope->thread().workerLoaderProxy(), taskMode, request, options, workerGlobalScope->url().strippedForUseAsReferrer(), workerGlobalScope->securityOrigin(), workerGlobalScope->contentSecurityPolicy()))
{
}
@@ -83,15 +83,13 @@
m_bridge.cancel();
}
-WorkerThreadableLoader::MainThreadBridge::MainThreadBridge(PassRefPtr<ThreadableLoaderClientWrapper> workerClientWrapper, WorkerLoaderProxy& loaderProxy, const String& taskMode,
+WorkerThreadableLoader::MainThreadBridge::MainThreadBridge(ThreadableLoaderClientWrapper& workerClientWrapper, WorkerLoaderProxy& loaderProxy, const String& taskMode,
const ResourceRequest& request, const ThreadableLoaderOptions& options, const String& outgoingReferrer,
const SecurityOrigin* securityOrigin, const ContentSecurityPolicy* contentSecurityPolicy)
- : m_workerClientWrapper(workerClientWrapper)
+ : m_workerClientWrapper(&workerClientWrapper)
, m_loaderProxy(loaderProxy)
, m_taskMode(taskMode.isolatedCopy())
{
- ASSERT(m_workerClientWrapper.get());
-
auto* requestData = request.copyData().release();
auto* optionsCopy = options.isolatedCopy().release();
Modified: trunk/Source/WebCore/loader/WorkerThreadableLoader.h (197550 => 197551)
--- trunk/Source/WebCore/loader/WorkerThreadableLoader.h 2016-03-04 07:41:45 UTC (rev 197550)
+++ trunk/Source/WebCore/loader/WorkerThreadableLoader.h 2016-03-04 08:27:36 UTC (rev 197551)
@@ -35,9 +35,6 @@
#include "ThreadableLoaderClient.h"
#include "ThreadableLoaderClientWrapper.h"
-#include <wtf/PassRefPtr.h>
-#include <wtf/RefCounted.h>
-#include <wtf/RefPtr.h>
#include <wtf/Threading.h>
#include <wtf/text/WTFString.h>
@@ -95,7 +92,7 @@
class MainThreadBridge : public ThreadableLoaderClient {
public:
// All executed on the worker context's thread.
- MainThreadBridge(PassRefPtr<ThreadableLoaderClientWrapper>, WorkerLoaderProxy&, const String& taskMode, const ResourceRequest&, const ThreadableLoaderOptions&, const String& outgoingReferrer, const SecurityOrigin*, const ContentSecurityPolicy*);
+ MainThreadBridge(ThreadableLoaderClientWrapper&, WorkerLoaderProxy&, const String& taskMode, const ResourceRequest&, const ThreadableLoaderOptions&, const String& outgoingReferrer, const SecurityOrigin*, const ContentSecurityPolicy*);
void cancel();
void destroy();
