Fix unified build in FIDO Pin.cpp
Created attachment 386831 [details] Patch
Created attachment 386832 [details] Patch
Comment on attachment 386832 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=386832&action=review Thanks for catching this bug. My bad. > Source/WebCore/Modules/webauthn/fido/DeviceResponseConverter.cpp:-47 > -constexpr size_t kResponseCodeLength = 1; I think it is better to name it CtapDeviceResponseCodeLength = 1 in FidoConstants.h after enum class CtapDeviceResponseCode.
Created attachment 386865 [details] Patch
Comment on attachment 386865 [details] Patch r=me. Thanks for making the patch.
Comment on attachment 386865 [details] Patch Clearing flags on attachment: 386865 Committed r254079: <https://trac.webkit.org/changeset/254079>
All reviewed patches have been landed. Closing bug.
<rdar://problem/58351406>