Changeset 285972 in webkit

Timestamp:

Nov 17, 2021, 7:05:08 PM (5 years ago)

Author:

Takashi.Komori@sony.com

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):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2021-11-17  Takashi Komori  <Takashi.Komori@sony.com>
+
+        [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  <wilander@apple.com>
 

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

@@ -222 +222 @@
 {
     m_multiHandle = curl_multi_init();
-
-    if (CurlContext::singleton().isHttp2Enabled())
-        curl_multi_setopt(m_multiHandle, CURLMOPT_PIPELINING, CURLPIPE_MULTIPLEX);
 }