On 2013/11/12 01:58:35, kpreid2 wrote:

https://codereview.appspot.com/24710043/diff/40001/tests/com/google/caja/plugin/test-scan-guest.js
File tests/com/google/caja/plugin/test-scan-guest.js (right):


https://codereview.appspot.com/24710043/diff/40001/tests/com/google/caja/plugin/test-scan-guest.js#newcode1491
tests/com/google/caja/plugin/test-scan-guest.js:1491: var specimen =
detach(Object.freeze({a: {}, 1: {}, '!': {}}));
On 2013/11/12 01:26:13, felix8a wrote:
> the frozenness is not preserved. detach only preserves isExtensible.
is that
> important?

It preserves frozenness because it preserves the extensible flag and
the
writable and configurable flags for each property. There is no
independent
frozen flag.

ah, right, duh. ok. still lgtm.

https://codereview.appspot.com/24710043/

--

--- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to