Bug 217559 – [WebAuthn] Implement a dummy WebAuthnProcess

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

[Jiewen Tan]

Implement a dummy WebAuthenticationAgent Part 2.

[Radar WebKit Bug Importer]

<rdar://problem/70168749>

[Jiewen Tan]

Created attachment 411008 [details]
Patch

[Jiewen Tan]

Created attachment 411378 [details]
Patch

[Jiewen Tan]

Created attachment 411396 [details]
Patch

[Brent Fulgham]

Comment on attachment 411396 [details]
Patch

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

You should have Chris Dumez take a look at the process assertion bits, but this seems to be correct as far as I can tell. You should also file a bug to add relevant entitlements to the new service once you start moving calls to USB/NFC things into this new code path.

> Source/WebKit/ChangeLog:25
> +        This patch therefore focues on making the WebAuthn process happen and removes the WebAuthn daemon.

focus

> Source/WebKit/ChangeLog:45
> +        Paperwork for introudcing the new WebAuthn process. Mostly copied from GPU process.

introducing

> Source/WebKit/ChangeLog:125
> +        Paperwork for introudcing the new WebAuthn process. Mostly copied from GPU process.

introducing

> Source/WebKit/ChangeLog:128
> +        The sandbox profile is originally from the GPU Process with IOKit related rules removed. Will tighten it again after the process is functional properly.

"process is fully functional"

> Source/WebKit/ChangeLog:153
> +        Paperwork for introudcing the new WebAuthn process. Mostly copied from GPU process.

introducing

> Source/WebCore/en.lproj/Localizable.strings:49
> +/* visible name of the GPU process. The argument is the application name. */

This isn't the GPU Process. This is the Web Authentication process!

> Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm:81
> +    // On Mac, leak a boost onto the NetworkProcess, GPUProcess and WebAuthnProcess.

Please use the "Oxford Comma":  "NetworkProcess, GPUProcess, and WebAuthnProcess."

[Jiewen Tan]

Comment on attachment 411396 [details]
Patch

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

Thanks Brent for r+ this patch.

>> Source/WebKit/ChangeLog:25
>> +        This patch therefore focues on making the WebAuthn process happen and removes the WebAuthn daemon.
> 
> focus

Fixed.

>> Source/WebKit/ChangeLog:45
>> +        Paperwork for introudcing the new WebAuthn process. Mostly copied from GPU process.
> 
> introducing

Fixed.

>> Source/WebKit/ChangeLog:125
>> +        Paperwork for introudcing the new WebAuthn process. Mostly copied from GPU process.
> 
> introducing

Fixed.

>> Source/WebKit/ChangeLog:128
>> +        The sandbox profile is originally from the GPU Process with IOKit related rules removed. Will tighten it again after the process is functional properly.
> 
> "process is fully functional"

Fixed.

>> Source/WebKit/ChangeLog:153
>> +        Paperwork for introudcing the new WebAuthn process. Mostly copied from GPU process.
> 
> introducing

Fixed.

>> Source/WebCore/en.lproj/Localizable.strings:49
>> +/* visible name of the GPU process. The argument is the application name. */
> 
> This isn't the GPU Process. This is the Web Authentication process!

Oops. Fixed.

>> Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm:81
>> +    // On Mac, leak a boost onto the NetworkProcess, GPUProcess and WebAuthnProcess.
> 
> Please use the "Oxford Comma":  "NetworkProcess, GPUProcess, and WebAuthnProcess."

Fixed.

[Jiewen Tan]

Created attachment 411602 [details]
Patch for landing

[EWS]

Committed r268605: <https://trac.webkit.org/changeset/268605>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 411602 [details].