Edgar, we should probably also clean up some of the code of https://hg.mozilla.org/mozilla-central/rev/91fe35e7df1fa3124c266c249ad0907320dfd956. E.g., Crypto::GetRandomValues will now be checked at the IDL layer so there's no need for it to throw inline, right? Similarly, the TypedArray bindings setup described there is no longer needed as we get safety from the IDL layer so we know if we can be dealing with shared beyond that and there's no need for two explicit APIs.
Bug 1575425 Comment 6 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Edgar, we should probably also clean up some of the code of https://hg.mozilla.org/mozilla-central/rev/91fe35e7df1fa3124c266c249ad0907320dfd956. E.g., Crypto::GetRandomValues will now be checked at the IDL layer so there's no need for it to throw inline, right? Similarly, the TypedArray bindings setup described there is no longer needed as we get safety from the IDL layer so we know if we can be dealing with shared beyond that and there's no need for two explicit APIs. (Edit: I guess an exception here is APIs that use `any`, such as `postMessage()`, but we need to check those either way as treating a SharedArrayBuffer as an empty buffer is wrong too.)