Re: StructuredCloneHelper

2015-09-10 Thread Andrea Marchesini
Yes, absolutely. I'll file a bug for this. On Tue, Sep 8, 2015 at 7:11 PM, Bobby Holley wrote: > Awesome, thanks for working to unify this stuff baku! > > Can this stuff be applied to StackScopedClone (used by Cu.cloneInto etc), > which currently does this stuff manually in ExportHelpers.cpp? >

Re: StructuredCloneHelper

2015-09-08 Thread Bobby Holley
Awesome, thanks for working to unify this stuff baku! Can this stuff be applied to StackScopedClone (used by Cu.cloneInto etc), which currently does this stuff manually in ExportHelpers.cpp? On Fri, Sep 4, 2015 at 1:12 AM, Andrea Marchesini wrote: > Hi all, > > In these days I landed quite a fe

Re: StructuredCloneHelper

2015-09-04 Thread Boris Zbarsky
On 9/4/15 4:12 AM, Andrea Marchesini wrote: 1. we had many postMessage() methods fully out of sync in terms of which clonable/transferable objects we were supporting. Now MessagePort, BroadcastChannel, window and worker and (partially) IPC share the same code base. Thank you for doing that! Th

Re: StructuredCloneHelper

2015-09-04 Thread Nicholas Nethercote
On Fri, Sep 4, 2015 at 6:12 PM, Andrea Marchesini wrote: > > In these days I landed quite a few patches to replace the use of > JSAutoStructuredCloneBuffer with something "better": StructuredCloneHelper. This might be a good time to mention bug 1106264, which is an OOM seemingly caused by JSAutoS