[webkit-reviews] review granted: [Bug 234533] Remove Web Authentication as experimental feature flag : [Attachment 447722] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Dec 21 09:21:58 PST 2021

Brent Fulgham <bfulgham at webkit.org> has granted j_pascoe at apple.com
<j_pascoe at apple.com>'s request for review:
Bug 234533: Remove Web Authentication as experimental feature flag

Attachment 447722: Patch


--- Comment #4 from Brent Fulgham <bfulgham at webkit.org> ---
Comment on attachment 447722
  --> https://bugs.webkit.org/attachment.cgi?id=447722

View in context: https://bugs.webkit.org/attachment.cgi?id=447722&action=review

r=me. So nice to see tons of code deleted!

> Source/WebKit/UIProcess/API/C/WKPreferencesRefPrivate.h:-503
> -WK_EXPORT bool WKPreferencesGetWebAuthenticationEnabled(WKPreferencesRef);

Ordinarily we would want to make this a stub and deprecate it, but I don't find
any uses of this call in other (non-WebKit) software so I think this is okay.

> +navigator.credentials is OK

I assume the WebAuthn bits of navigator.credentials aren't present in WK1, but
the top-level namespace exists?

More information about the webkit-reviews mailing list