Bug 208533 – [WebAuthn] Implement -[_WKWebAuthenticationPanelDelegate panel:decidePolicyForLocalAuthenticatorWithCompletionHandler:] SPI

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Jiewen Tan]

Implement -[_WKWebAuthenticationPanelDelegate panel:decidePolicyForLocalAuthenticatorWithCompletionHandler:] SPI.

[Radar WebKit Bug Importer]

<rdar://problem/60010184>

[Jiewen Tan]

Created attachment 392320 [details]
Patch

[Jiewen Tan]

Comment on attachment 392320 [details]
Patch

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

> Source/WebKit/UIProcess/API/Cocoa/_WKWebAuthenticationPanel.h:93
> +@property (nonatomic, readonly, copy) NSSet *transports;

Should mention this change in the change log as well.

[Jiewen Tan]

Created attachment 392325 [details]
Patch

[Jiewen Tan]

Created attachment 392333 [details]
Patch

[Alex Christensen]

Comment on attachment 392333 [details]
Patch

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

> Source/WebCore/platform/LocalizedStrings.cpp:1213
> +    return WEB_UI_STRING("Touch ID to sign into this website.", "Use Touch ID to sign into this website");

What about Face ID?

> Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalConnection.h:53
> +    enum class UserVerification {

: bool

[Jiewen Tan]

Comment on attachment 392333 [details]
Patch

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

Thanks Alex for the review.

>> Source/WebCore/platform/LocalizedStrings.cpp:1213
>> +    return WEB_UI_STRING("Touch ID to sign into this website.", "Use Touch ID to sign into this website");
> 
> What about Face ID?

FaceID prompt doesn't show any title/message strings.

>> Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalConnection.h:53
>> +    enum class UserVerification {
> 
> : bool

Fixed.

[Jiewen Tan]

Created attachment 392446 [details]
Patch

[Alex Christensen]

Comment on attachment 392333 [details]
Patch

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

> Source/WebKit/UIProcess/API/APIWebAuthenticationPanelClient.h:54
> +    // FIXME: Ideally, LocalAuthenticatorPolicy::Disallow should be used. This is a quirk to bypass C API for WebKitTestRunner.

Let's not commit this.  Make the C API.  The default should be safe.
You don't have to make it fully asynchronous with a listener object and new object type, but please add a new C API client type with this callback that returns a bool, then WebKitTestRunner's implementation will return true and the default will be false.

[Jiewen Tan]

(In reply to Alex Christensen from comment #9)
> Comment on attachment 392333 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=392333&action=review
> 
> > Source/WebKit/UIProcess/API/APIWebAuthenticationPanelClient.h:54
> > +    // FIXME: Ideally, LocalAuthenticatorPolicy::Disallow should be used. This is a quirk to bypass C API for WebKitTestRunner.
> 
> Let's not commit this.  Make the C API.  The default should be safe.
> You don't have to make it fully asynchronous with a listener object and new
> object type, but please add a new C API client type with this callback that
> returns a bool, then WebKitTestRunner's implementation will return true and
> the default will be false.

Fixed.

[Jiewen Tan]

Created attachment 392455 [details]
Patch

[Jiewen Tan]

(In reply to Jiewen Tan from comment #11)
> Created attachment 392455 [details]
> Patch

Thanks, Alex.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 392455 [details]:

editing/spelling/spellcheck-paste-continuous-disabled.html bug 208016 (authors: [e-mail] and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 392455 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: [e-mail], [e-mail], and [e-mail])
svg/custom/object-sizing-explicit-height.xhtml bug 208592 (author: [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 392455 [details]
Patch

Rejecting attachment 392455 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 392455, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 5000 characters of output:
coa/LocalAuthenticator.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalConnection.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalConnection.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalService.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/WebAuthenticationPanelClient.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/WebAuthenticationPanelClient.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Mock/MockLocalConnection.h
	M	Source/WebKit/UIProcess/WebAuthentication/Mock/MockLocalConnection.mm
	M	Source/WebKit/UIProcess/WebAuthentication/WebAuthenticationFlags.h
	M	Tools/ChangeLog
	M	Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
	M	Tools/TestWebKitAPI/Tests/WebKitCocoa/_WKWebAuthenticationPanel.mm

ERROR from SVN:
Merge conflict during commit: Conflict at '/trunk/Source/WebKit/ChangeLog'
W: 937f80b4205f4987cb9a9d2ec8958d431ccbf351 and refs/remotes/origin/master differ, using rebase:
:040000 040000 a84a575ecae8477dd96f98fad91114e27723d018 6109000d1db81b6f0d5dc73d51b1fdea817f4f32 M	LayoutTests
:040000 040000 81cb632c531721fd1a234479ec9a2a896999554c 531e3b2f1c1c5c8c6b674937f3171e70ba8c7fa6 M	Source
:040000 040000 5443204dbe643da13513ba0c30cbe2ebd8a36bd2 1cbfe2da399663568c33343d7e846427f6383142 M	Tools
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	D	Tools/TestWebKitAPI/Tests/WebKitCocoa/web-authentication-get-assertion-la.html
	M	LayoutTests/http/wpt/webauthn/public-key-credential-create-failure-local.https.html
	M	LayoutTests/http/wpt/webauthn/public-key-credential-get-failure-local.https.html
	M	Source/WebCore/ChangeLog
	M	Source/WebCore/en.lproj/Localizable.strings
	M	Source/WebCore/platform/LocalizedStrings.cpp
	M	Source/WebCore/platform/LocalizedStrings.h
	M	Source/WebKit/ChangeLog
	M	Source/WebKit/Platform/spi/Cocoa/LocalAuthenticationSPI.h
	M	Source/WebKit/UIProcess/API/APIWebAuthenticationPanelClient.h
	M	Source/WebKit/UIProcess/API/C/WKPage.cpp
	M	Source/WebKit/UIProcess/API/Cocoa/_WKWebAuthenticationPanel.h
	M	Source/WebKit/UIProcess/API/Cocoa/_WKWebAuthenticationPanel.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Authenticator.h
	M	Source/WebKit/UIProcess/WebAuthentication/AuthenticatorManager.cpp
	M	Source/WebKit/UIProcess/WebAuthentication/AuthenticatorManager.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalConnection.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalConnection.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalService.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/WebAuthenticationPanelClient.h
	M	Source/WebKit/UIProcess/WebAuthentication/Cocoa/WebAuthenticationPanelClient.mm
	M	Source/WebKit/UIProcess/WebAuthentication/Mock/MockLocalConnection.h
	M	Source/WebKit/UIProcess/WebAuthentication/Mock/MockLocalConnection.mm
	M	Source/WebKit/UIProcess/WebAuthentication/WebAuthenticationFlags.h
	M	Tools/ChangeLog
	M	Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
	M	Tools/TestWebKitAPI/Tests/WebKitCocoa/_WKWebAuthenticationPanel.mm

ERROR from SVN:
Merge conflict during commit: Conflict at '/trunk/Source/WebKit/ChangeLog'
W: 937f80b4205f4987cb9a9d2ec8958d431ccbf351 and refs/remotes/origin/master differ, using rebase:
:040000 040000 a84a575ecae8477dd96f98fad91114e27723d018 6109000d1db81b6f0d5dc73d51b1fdea817f4f32 M	LayoutTests
:040000 040000 81cb632c531721fd1a234479ec9a2a896999554c 531e3b2f1c1c5c8c6b674937f3171e70ba8c7fa6 M	Source
:040000 040000 5443204dbe643da13513ba0c30cbe2ebd8a36bd2 1cbfe2da399663568c33343d7e846427f6383142 M	Tools
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
From https://git.webkit.org/git/WebKit
   0a6a1caeff1..19142dbcd32  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 257873 = 0a6a1caeff1415c65c45148a6b570c2e464b64c3
r257874 = 26e4328157b5c42de5d895971532024d34f9c9a9
r257875 = 19142dbcd32c2c760f38ad4fc7c0023f88f72d7a
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: https://webkit-queues.webkit.org/results/13333637

[Jiewen Tan]

Committed r257877: <https://trac.webkit.org/changeset/257877>

[Jiewen Tan]

A build fix:
Committed r257891: <https://trac.webkit.org/changeset/257891>