Changeset 278783 in webkit

Timestamp:

Jun 11, 2021, 2:24:13 PM (5 years ago)

Author:

commit-queue@webkit.org

Message:

Don't include certificate info in WebURLSchemeTask::didReceiveResponse
​https://bugs.webkit.org/show_bug.cgi?id=226939
<rdar://79178335>

Patch by Alex Christensen <​achristensen@webkit.org> on 2021-06-11
Reviewed by Chris Dumez.

This call was introduced in r215384 but is unused since WKURLSchemeTask isn't used for HTTPS.
It is causing hangs sometimes, so let's drop it.

• UIProcess/WebURLSchemeTask.cpp:

(WebKit::WebURLSchemeTask::didReceiveResponse):

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/WebURLSchemeTask.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2021-06-11  Alex Christensen  <achristensen@webkit.org>
+
+        Don't include certificate info in WebURLSchemeTask::didReceiveResponse
+        https://bugs.webkit.org/show_bug.cgi?id=226939
+        <rdar://79178335>
+
+        Reviewed by Chris Dumez.
+
+        This call was introduced in r215384 but is unused since WKURLSchemeTask isn't used for HTTPS.
+        It is causing hangs sometimes, so let's drop it.
+
+        * UIProcess/WebURLSchemeTask.cpp:
+        (WebKit::WebURLSchemeTask::didReceiveResponse):
+
 2021-06-11  Jonathan Bedard  <jbedard@apple.com>
 

trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp

@@ -162 +162 @@
     m_responseSent = true;
 
-    response.includeCertificateInfo();
-
     if (isSync())
         m_syncResponse = response;