Summary: | [WebAuthn] Make CtapHidAuthenticator/U2fHidAuthenticator to CtapAuthenticator/U2fAuthenticator | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jiewen Tan <jiewen_tan> | ||||||||
Component: | WebCore Misc. | Assignee: | Jiewen Tan <jiewen_tan> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | alex.gaynor, bfulgham, cdumez, jiewen_tan, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 181943 | ||||||||||
Attachments: |
|
Description
Jiewen Tan
2018-11-11 18:11:38 PST
Created attachment 376125 [details]
Patch
Created attachment 376134 [details]
Patch
Created attachment 376138 [details]
Patch
Comment on attachment 376138 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376138&action=review > Source/WebKit/ChangeLog:9 > + This patch makes a ABC CtapDriver, which suffers as an interface for CtapAuthenticator/U2fAuthenticator to use, suffers => services Comment on attachment 376138 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376138&action=review >> Source/WebKit/ChangeLog:9 >> + This patch makes a ABC CtapDriver, which suffers as an interface for CtapAuthenticator/U2fAuthenticator to use, > > suffers => services I do not understand this sentence, even after your typo fix. Comment on attachment 376138 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376138&action=review Thanks Chris for r+ this patch. >>> Source/WebKit/ChangeLog:9 >>> + This patch makes a ABC CtapDriver, which suffers as an interface for CtapAuthenticator/U2fAuthenticator to use, >> >> suffers => services > > I do not understand this sentence, even after your typo fix. I would expand the sentence to include a description of the CtapDriver. The CtapDriver is an abstract interface for all drivers that implements the CTAP protocol for different transports, i.e. HID, NFC, BLE ...... Committed r248631: <https://trac.webkit.org/changeset/248631> |