Closed Bug 1415675 Opened 7 years ago Closed 7 years ago

Web Authentication - Support AbortSignal types

Categories

(Core :: DOM: Device Interfaces, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Future
Tracking Status
firefox58 --- wontfix
firefox59 --- fixed

People

(Reporter: jcj, Assigned: ttaubert)

References

Details

(Whiteboard: [webauthn][webauthn-wd07])

Attachments

(1 file)

The Create and Get methods need to accept an AbortController, and 1) call the callback, if any, upon an abort, and 2) send a cancel (like we do with a tab switch) when the abort is called by Javascript.
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
OS: Mac OS X → All
Hardware: x86 → All
Blocks: 1388854
Comment on attachment 8928427 [details] Bug 1415675 - Web Authentication - Support AbortSignal types r=jcj J.C. Jones [:jcj] has approved the revision. https://phabricator.services.mozilla.com/D245#5917
Attachment #8928427 - Flags: review+
I'm honestly a little surprised to see no failures. Well, let's give it a shot and push it.
No, let's wait for a DOM peer to r+. Sorry ;)
Comment on attachment 8928427 [details] Bug 1415675 - Web Authentication - Support AbortSignal types r=jcj J.C. Jones [:jcj] has been removed from the revision. https://phabricator.services.mozilla.com/D245#5967
Attachment #8928427 - Flags: review+
Comment on attachment 8928427 [details] Bug 1415675 - Web Authentication - Support AbortSignal types r=jcj J.C. Jones [:jcj] has approved the revision. https://phabricator.services.mozilla.com/D245#6006
Attachment #8928427 - Flags: review+
Attachment #8928427 - Flags: review?(bugs)
Comment on attachment 8928427 [details] Bug 1415675 - Web Authentication - Support AbortSignal types r=jcj Olli Pettay [:smaug] has approved the revision. https://phabricator.services.mozilla.com/D245#6007
Attachment #8928427 - Flags: review+
Pushed by ttaubert@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/805939626996 Web Authentication - Support AbortSignal types r=jcj,smaug
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: