Need to get bug 856338 fixed first.

Not super nice, but hopefully we can get rid of this event once Audio API works well enough. https://tbpl.mozilla.org/?tree=Try&rev=79e5a32df4b4

Comment on attachment 741040 [details] [diff] [review] patch Review of attachment 741040 [details] [diff] [review]: ----------------------------------------------------------------- ::: content/events/src/nsDOMNotifyAudioAvailableEvent.cpp @@ +143,5 @@ > + } > + > + nsAutoArrayPtr<float> buffer; > + if (!aFrameBuffer.IsNull()) { > + buffer = new float[aFrameBufferLength]; Hmm, don't we want this to be a fallible allocation? Seems like this could be controlled from script? ::: content/events/src/nsDOMNotifyAudioAvailableEvent.h @@ +39,5 @@ > float aTime); > > ~nsDOMNotifyAudioAvailableEvent(); > > + virtual JSObject* WrapObject(JSContext* aCx, JSObject* aScope) I suppose I should have commented about that in all the previous conversions :-/: MOZ_OVERRIDE. ::: dom/webidl/NotifyAudioAvailableEvent.webidl @@ +6,5 @@ > + > +interface NotifyAudioAvailableEvent : Event > +{ > + [Throws] > + readonly attribute any frameBuffer; This really should return Float32Array.

(In reply to Peter Van der Beken [:peterv] from comment #3) > > + [Throws] > > + readonly attribute any frameBuffer; > > This really should return Float32Array. But our current infrastructure for TypedArrays doesn't make it easy to cache the JS.

Ah, I was told the getter becomes JSObject*