Title: [221081] trunk/Source/WebKit
- Revision
- 221081
- Author
- [email protected]
- Date
- 2017-08-23 10:14:09 -0700 (Wed, 23 Aug 2017)
Log Message
Regression(r221059): NetworkDataTask::didReceiveResponse() should not use PolicyUse for HTTP/0.9
https://bugs.webkit.org/show_bug.cgi?id=175872
Reviewed by Daniel Bates.
r221059 was calling the ResponseCompletionHandler with {} which ended up being
PolicyUse. Since this is an error case and do not want to receive the data, it
makes more sense to use PolicyIgnore instead. There should not be a behavior
change on WebKit side though since we are cancelling the load right after
calling the completion handler anyway.
Tests under http/tests/security/http-0.9/ are still passing.
* NetworkProcess/NetworkDataTask.cpp:
(WebKit::NetworkDataTask::didReceiveResponse):
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (221080 => 221081)
--- trunk/Source/WebKit/ChangeLog 2017-08-23 17:05:33 UTC (rev 221080)
+++ trunk/Source/WebKit/ChangeLog 2017-08-23 17:14:09 UTC (rev 221081)
@@ -1,3 +1,21 @@
+2017-08-23 Chris Dumez <[email protected]>
+
+ Regression(r221059): NetworkDataTask::didReceiveResponse() should not use PolicyUse for HTTP/0.9
+ https://bugs.webkit.org/show_bug.cgi?id=175872
+
+ Reviewed by Daniel Bates.
+
+ r221059 was calling the ResponseCompletionHandler with {} which ended up being
+ PolicyUse. Since this is an error case and do not want to receive the data, it
+ makes more sense to use PolicyIgnore instead. There should not be a behavior
+ change on WebKit side though since we are cancelling the load right after
+ calling the completion handler anyway.
+
+ Tests under http/tests/security/http-0.9/ are still passing.
+
+ * NetworkProcess/NetworkDataTask.cpp:
+ (WebKit::NetworkDataTask::didReceiveResponse):
+
2017-08-22 Tim Horton <[email protected]>
_WKThumbnailView should use the screen color space instead of sRGB
Modified: trunk/Source/WebKit/NetworkProcess/NetworkDataTask.cpp (221080 => 221081)
--- trunk/Source/WebKit/NetworkProcess/NetworkDataTask.cpp 2017-08-23 17:05:33 UTC (rev 221080)
+++ trunk/Source/WebKit/NetworkProcess/NetworkDataTask.cpp 2017-08-23 17:14:09 UTC (rev 221081)
@@ -103,7 +103,7 @@
auto url = ""
std::optional<uint16_t> port = url.port();
if (port && !isDefaultPortForProtocol(port.value(), url.protocol())) {
- completionHandler({ });
+ completionHandler(PolicyIgnore);
cancel();
m_client->didCompleteWithError({ String(), 0, url, "Cancelled load from '" + url.stringCenterEllipsizedToLength() + "' because it is using HTTP/0.9." });
return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
