| Summary: | [WebAuthn] Clean up WebAuthenticationModern and WebAuthnProcess | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | pascoe <pascoe> | ||||
| Component: | WebKit Misc. | Assignee: | pascoe <pascoe> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | bfulgham, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=239073 | ||||||
| Attachments: |
|
||||||
|
Description
pascoe@apple.com
2022-05-11 14:45:21 PDT
Created attachment 459183 [details]
Patch
Comment on attachment 459183 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=459183&action=review So much cleanup! r=me > Tools/ChangeLog:-1977 > - (TestWebKitAPI::WebCore::webAuthenticationModernExperimentalFeature): Deleted. We don't need this ChangeLog comment removal > Tools/ChangeLog:2011 > + (TestWebKitAPI::WebCore::webAuthenticationModernExperimentalFeature): Deleted. This seems to be in the wrong part of the file (is perhaps a dupe of the 5/11 comment) > Tools/TestWebKitAPI/Tests/WebKitCocoa/_WKWebAuthenticationPanel.mm:-1437 > -} Are these two tests really not needed anymore? It doesn't seem like disallowing the local authenticator is specifically related to the WebAuthn process, is it? It doesn't seem like the Mac-AS-debug-wk2 failures are related. I have re-ran them. (In reply to Brent Fulgham from comment #3) > > Tools/TestWebKitAPI/Tests/WebKitCocoa/_WKWebAuthenticationPanel.mm:-1437 > > -} > > Are these two tests really not needed anymore? It doesn't seem like > disallowing the local authenticator is specifically related to the WebAuthn > process, is it? These tests are from when the WebAuthenticationPanel used a UI delegate method (decidePolicyForLocalAuthenticator) to ask if using the platform authenticator is permitted. This is superseded by requestLAContextForUserVerification, which effectively does the same thing, but does not return ExceptionData { UnknownError, "Disallow local authenticator."_s } (the standard request cancelled error is used). Pull request: https://github.com/WebKit/WebKit/pull/744 Committed r294438 (250716@main): <https://commits.webkit.org/250716@main> Reviewed commits have been landed. Closing PR #744 and removing active labels. |