[Webkit-unassigned] [Bug 232977] _WKWebAuthenticationPanel should expose a way to encode CTAP commands

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Nov 12 14:12:25 PST 2021


David Kilzer (:ddkilzer) <ddkilzer at webkit.org> changed:

           What    |Removed                     |Added
 Attachment #444029|review?                     |review+
              Flags|                            |

--- Comment #10 from David Kilzer (:ddkilzer) <ddkilzer at webkit.org> ---
Comment on attachment 444029
  --> https://bugs.webkit.org/attachment.cgi?id=444029

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

Let me know if you need cq+ as well.

> Source/WebKit/UIProcess/API/Cocoa/_WKWebAuthenticationPanel.mm:648
> +    RetainPtr<NSData> clientDataJSON;
> +
> +    clientDataJSON = produceClientDataJson(type, challenge, origin);
> +#endif
> +
> +    return clientDataJSON.autorelease();

Another way to write this would be:

    return produceClientDataJson(type, challenge, origin).autorelease();
    return nullptr;

This does NOT need to be changed to land the patch, but it makes it clearer what the code path does when ENABLE(WEB_AUTHN) evaluates to false.

(Similar change could be done for the other methods, too, but no need to change this to land it.)

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20211112/92aca0cd/attachment.htm>

More information about the webkit-unassigned mailing list