Closed Bug 1363029 Opened 3 years ago Closed 3 years ago
ICrypto Hash used off main thread - must be builtinclass
the url-classifier uses nsICryptoHash in a worker thread.. that means it can't have a js implementation (you can still use it from js). https://dxr.mozilla.org/mozilla-central/source/toolkit/components/url-classifier/nsUrlClassifierDBService.cpp#963 this patch marks it as a builtinclass to enforce that.
Comment on attachment 8865439 [details] Bug 1363029 - mark nsICryptoHash builtinclass https://reviewboard.mozilla.org/r/137112/#review140218 LGTM. I suppose we should audit other PSM interfaces for this sort of thing...
Attachment #8865439 - Flags: review?(dkeeler) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/1ee4283fc4df mark nsICryptoHash builtinclass r=keeler
You need to log in before you can comment on or make changes to this bug.