Title: [285972] trunk/Source/WebCore
Revision
285972
Author
[email protected]
Date
2021-11-17 19:05:08 -0800 (Wed, 17 Nov 2021)

Log Message

[Curl] Remove unnecessary option setting
https://bugs.webkit.org/show_bug.cgi?id=233249

Reviewed by Fujii Hironori.

Remove setting CURLPIPE_MULTIPLEX option.
The option is enabled by default since curl 7.62.0 [1].

[1] https://curl.se/libcurl/c/curl_multi_setopt.html

No new tests, covered by existing tests.

* platform/network/curl/CurlContext.cpp:
(WebCore::CurlMultiHandle::CurlMultiHandle):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (285971 => 285972)


--- trunk/Source/WebCore/ChangeLog	2021-11-18 02:47:45 UTC (rev 285971)
+++ trunk/Source/WebCore/ChangeLog	2021-11-18 03:05:08 UTC (rev 285972)
@@ -1,3 +1,20 @@
+2021-11-17  Takashi Komori  <[email protected]>
+
+        [Curl] Remove unnecessary option setting
+        https://bugs.webkit.org/show_bug.cgi?id=233249
+
+        Reviewed by Fujii Hironori.
+
+        Remove setting CURLPIPE_MULTIPLEX option.
+        The option is enabled by default since curl 7.62.0 [1].
+
+        [1] https://curl.se/libcurl/c/curl_multi_setopt.html
+
+        No new tests, covered by existing tests.
+
+        * platform/network/curl/CurlContext.cpp:
+        (WebCore::CurlMultiHandle::CurlMultiHandle):
+
 2021-11-17  John Wilander  <[email protected]>
 
         PCM: Add capability for click destination to fire triggering event without cross-site requests to the click source

Modified: trunk/Source/WebCore/platform/network/curl/CurlContext.cpp (285971 => 285972)


--- trunk/Source/WebCore/platform/network/curl/CurlContext.cpp	2021-11-18 02:47:45 UTC (rev 285971)
+++ trunk/Source/WebCore/platform/network/curl/CurlContext.cpp	2021-11-18 03:05:08 UTC (rev 285972)
@@ -221,9 +221,6 @@
 CurlMultiHandle::CurlMultiHandle()
 {
     m_multiHandle = curl_multi_init();
-
-    if (CurlContext::singleton().isHttp2Enabled())
-        curl_multi_setopt(m_multiHandle, CURLMOPT_PIPELINING, CURLPIPE_MULTIPLEX);
 }
 
 CurlMultiHandle::~CurlMultiHandle()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to