Changeset 219265 in webkit

Timestamp:

Jul 7, 2017 1:20:24 PM (7 years ago)

Author:

commit-queue@webkit.org

Message:

[Curl] Remove data url handler for async load
​https://bugs.webkit.org/show_bug.cgi?id=174263

data url is handled by ResourceLoader. No need for specific handling
in platform dependent layer.

Patch by Basuke Suzuki <Basuke Suzuki> on 2017-07-07
Reviewed by Alex Christensen.

• platform/network/curl/ResourceHandleManager.cpp:

(WebCore::ResourceHandleManager::startJob):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/network/curl/ResourceHandleManager.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-07-07  Basuke Suzuki  <Basuke.Suzuki@sony.com>
+
+        [Curl] Remove data url handler for async load
+        https://bugs.webkit.org/show_bug.cgi?id=174263
+
+        data url is handled by ResourceLoader. No need for specific handling
+        in platform dependent layer.
+
+        Reviewed by Alex Christensen.
+
+        * platform/network/curl/ResourceHandleManager.cpp:
+        (WebCore::ResourceHandleManager::startJob):
+
 2017-07-07  Matt Lewis  <jlewis3@apple.com>
 

trunk/Source/WebCore/platform/network/curl/ResourceHandleManager.cpp

@@ -225 +225 @@
 void ResourceHandleManager::startJob(ResourceHandle* job)
 {
-    URL url = job->firstRequest().url();
-
-    if (url.protocolIsData()) {
-        job->handleDataURL();
-        job->deref();
-        return;
-    }
-
     job->initialize();