Title: [278783] trunk/Source/WebKit
Revision
278783
Author
[email protected]
Date
2021-06-11 14:24:13 -0700 (Fri, 11 Jun 2021)

Log Message

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

Patch by Alex Christensen <[email protected]> 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):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (278782 => 278783)


--- trunk/Source/WebKit/ChangeLog	2021-06-11 21:21:31 UTC (rev 278782)
+++ trunk/Source/WebKit/ChangeLog	2021-06-11 21:24:13 UTC (rev 278783)
@@ -1,3 +1,17 @@
+2021-06-11  Alex Christensen  <[email protected]>
+
+        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  <[email protected]>
 
         [Monterey] Support building WebKit

Modified: trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp (278782 => 278783)


--- trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp	2021-06-11 21:21:31 UTC (rev 278782)
+++ trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp	2021-06-11 21:24:13 UTC (rev 278783)
@@ -161,8 +161,6 @@
 
     m_responseSent = true;
 
-    response.includeCertificateInfo();
-
     if (isSync())
         m_syncResponse = response;
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to