Make WebTaskRunner ThreadSafeRefCounted

third_party/WebKit/Source/core/fetch/ResourceLoader.cpp

Index: third_party/WebKit/Source/core/fetch/ResourceLoader.cpp
diff --git a/third_party/WebKit/Source/core/fetch/ResourceLoader.cpp b/third_party/WebKit/Source/core/fetch/ResourceLoader.cpp
index 1dc79fd6d6d80020d1ff43d6eb7032e40e53be56..2c97c02871e074ce5f9e4ee7ae90f441a580e47f 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceLoader.cpp
+++ b/third_party/WebKit/Source/core/fetch/ResourceLoader.cpp
@@ -72,7 +72,7 @@ DEFINE_TRACE(ResourceLoader) {
 }
 
 void ResourceLoader::start(const ResourceRequest& request,
-                           WebTaskRunner* loadingTaskRunner,
+                           RefPtr<WebTaskRunner> loadingTaskRunner,
                            bool defersLoading) {
   DCHECK(!m_loader);
   if (m_resource->options().synchronousPolicy == RequestSynchronously &&
@@ -84,7 +84,7 @@ void ResourceLoader::start(const ResourceRequest& request,
   m_loader = wrapUnique(Platform::current()->createURLLoader());
   DCHECK(m_loader);
   m_loader->setDefersLoading(defersLoading);
-  m_loader->setLoadingTaskRunner(loadingTaskRunner);
+  m_loader->setLoadingTaskRunner(loadingTaskRunner.get());

[haraken, 2016/12/08 01:49:02]

Why do we need get()?

[tzik, 2016/12/12 12:18:52]

eOn 2016/12/08 01:49:02, haraken wrote:
Since WebURLLoader is in public/platform, we are using a raw pointer there, so
we need a conversion from the RefPtr to the raw pointer.

 
   if (m_isCacheAwareLoadingActivated) {
     // Override cache policy for cache-aware loading. If this request fails, a
@@ -103,11 +103,11 @@ void ResourceLoader::start(const ResourceRequest& request,
 }
 
 void ResourceLoader::restart(const ResourceRequest& request,
-                             WebTaskRunner* loadingTaskRunner,
+                             RefPtr<WebTaskRunner> loadingTaskRunner,
                              bool defersLoading) {
   CHECK_EQ(m_resource->options().synchronousPolicy, RequestAsynchronously);
   m_loader.reset();
-  start(request, loadingTaskRunner, defersLoading);
+  start(request, std::move(loadingTaskRunner), defersLoading);
 }
 
 void ResourceLoader::setDefersLoading(bool defers) {