Closed Bug 1407565 Opened 7 years ago Closed 7 years ago

Add NS_IsMainThread() assertions to WebAuthnManager

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: ttaubert, Assigned: ttaubert)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

We should assert, just for correctness, that we're on the main thread most (all?) of the time in WebAuthManager methods.
Comment on attachment 8917509 [details] Bug 1407565 - Add NS_IsMainThread() assertions to WebAuthnManager r=jcj J.C. Jones [:jcj] has approved the revision. https://phabricator.services.mozilla.com/D116#2588
Attachment #8917509 - Flags: review+
Pushed by ttaubert@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/fc50cd79955f Add NS_IsMainThread() assertions to WebAuthnManager r=jcj
Priority: -- → P2
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: