Title: [236238] tags/Safari-607.1.7/Source/WebKit
- Revision
- 236238
- Author
- [email protected]
- Date
- 2018-09-19 20:41:36 -0700 (Wed, 19 Sep 2018)
Log Message
Cherry-pick r236232. rdar://problem/44620769
REGRESSION(236154) C API clients not using WKPageSetPageNavigationClient couldn't complete HTTPS requests
https://bugs.webkit.org/show_bug.cgi?id=189771
Reviewed by Tim Horton.
* UIProcess/API/APINavigationClient.h:
(API::NavigationClient::didReceiveAuthenticationChallenge):
Perform default behavior if there's an authentication challenge but we're using the default navigation client.
I wish we had infrastructure to test this but WebKitTestRunner uses WKPageSetNavigationClient
and we don't have the ability to do networking from API tests. This is a growing problem I intend to solve.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@236232 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: tags/Safari-607.1.7/Source/WebKit/ChangeLog (236237 => 236238)
--- tags/Safari-607.1.7/Source/WebKit/ChangeLog 2018-09-20 02:31:42 UTC (rev 236237)
+++ tags/Safari-607.1.7/Source/WebKit/ChangeLog 2018-09-20 03:41:36 UTC (rev 236238)
@@ -1,3 +1,35 @@
+2018-09-19 Kocsen Chung <[email protected]>
+
+ Cherry-pick r236232. rdar://problem/44620769
+
+ REGRESSION(236154) C API clients not using WKPageSetPageNavigationClient couldn't complete HTTPS requests
+ https://bugs.webkit.org/show_bug.cgi?id=189771
+
+ Reviewed by Tim Horton.
+
+ * UIProcess/API/APINavigationClient.h:
+ (API::NavigationClient::didReceiveAuthenticationChallenge):
+ Perform default behavior if there's an authentication challenge but we're using the default navigation client.
+ I wish we had infrastructure to test this but WebKitTestRunner uses WKPageSetNavigationClient
+ and we don't have the ability to do networking from API tests. This is a growing problem I intend to solve.
+
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@236232 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2018-09-19 Alex Christensen <[email protected]>
+
+ REGRESSION(236154) C API clients not using WKPageSetPageNavigationClient couldn't complete HTTPS requests
+ https://bugs.webkit.org/show_bug.cgi?id=189771
+
+ Reviewed by Tim Horton.
+
+ * UIProcess/API/APINavigationClient.h:
+ (API::NavigationClient::didReceiveAuthenticationChallenge):
+ Perform default behavior if there's an authentication challenge but we're using the default navigation client.
+ I wish we had infrastructure to test this but WebKitTestRunner uses WKPageSetNavigationClient
+ and we don't have the ability to do networking from API tests. This is a growing problem I intend to solve.
+
2018-09-18 Brent Fulgham <[email protected]>
[iOS] Allow WebContent process to check the "Protocol Characteristics" of files to which it has access
Modified: tags/Safari-607.1.7/Source/WebKit/UIProcess/API/APINavigationClient.h (236237 => 236238)
--- tags/Safari-607.1.7/Source/WebKit/UIProcess/API/APINavigationClient.h 2018-09-20 02:31:42 UTC (rev 236237)
+++ tags/Safari-607.1.7/Source/WebKit/UIProcess/API/APINavigationClient.h 2018-09-20 03:41:36 UTC (rev 236238)
@@ -27,6 +27,8 @@
#include "APIData.h"
#include "APIString.h"
+#include "AuthenticationChallengeProxy.h"
+#include "AuthenticationDecisionListener.h"
#include "PluginModuleInfo.h"
#include "ProcessTerminationReason.h"
#include "SameDocumentNavigationType.h"
@@ -85,8 +87,7 @@
virtual void renderingProgressDidChange(WebKit::WebPageProxy&, WebCore::LayoutMilestones) { }
- virtual bool canAuthenticateAgainstProtectionSpace(WebKit::WebPageProxy&, WebKit::WebProtectionSpace*) { return false; }
- virtual void didReceiveAuthenticationChallenge(WebKit::WebPageProxy&, WebKit::AuthenticationChallengeProxy&) { }
+ virtual void didReceiveAuthenticationChallenge(WebKit::WebPageProxy&, WebKit::AuthenticationChallengeProxy& challenge) { challenge.listener().performDefaultHandling(); }
// FIXME: These function should not be part of this client.
virtual bool processDidTerminate(WebKit::WebPageProxy&, WebKit::ProcessTerminationReason) { return false; }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
