Re: WaveShaperNode.curve

On Mon, May 13, 2013 at 9:20 AM, Ehsan Akhgari <ehsan.akhgari@gmail.com>wrote:

> The current WebKit implementation of this node is racy, since the
> processing code only protects against simultaneous setting of the curve
> property, not against modifying the contents of the ArrayBuffer.
>
> In the Gecko implementation, I'm just copying the contents of the array
> upon setting curve for now, but I think a better fix would be to neuter the
> contents of the array, and provide a copy of the original contents of the
> array if contents reads the curve property again.
>
> Does this make sense?
>

Sounds good to me of course :-)

Rob
-- 
q“qIqfq qyqoquq qlqoqvqeq qtqhqoqsqeq qwqhqoq qlqoqvqeq qyqoquq,q qwqhqaqtq
qcqrqeqdqiqtq qiqsq qtqhqaqtq qtqoq qyqoquq?q qEqvqeqnq qsqiqnqnqeqrqsq
qlqoqvqeq qtqhqoqsqeq qwqhqoq qlqoqvqeq qtqhqeqmq.q qAqnqdq qiqfq qyqoquq
qdqoq qgqoqoqdq qtqoq qtqhqoqsqeq qwqhqoq qaqrqeq qgqoqoqdq qtqoq qyqoquq,q
qwqhqaqtq qcqrqeqdqiqtq qiqsq qtqhqaqtq qtqoq qyqoquq?q qEqvqeqnq
qsqiqnqnqeqrqsq qdqoq qtqhqaqtq.q"