about halfway through the test-scan-guest changes
https://codereview.appspot.com/13024043/diff/1/tests/com/google/caja/plugin/test-scan-guest.js
File tests/com/google/caja/plugin/test-scan-guest.js (right):
https://codereview.appspot.com/13024043/diff/1/tests/com/google/caja/plugin/test-scan-guest.js#newcode412
tests/com/google/caja/plugin/test-scan-guest.js:412: forEach:
function(f) {
this seems to me a very weird place to have a forEach. It ends up being
called only once, and it's not immediately obvious why other types of
Refs don't have a forEach, or why Ref.is does have a forEach.
maybe RefAnyFrame should be split into two functions, one that returns
an Array and another that returns a Ref based on the Array?
https://codereview.appspot.com/13024043/
--
---
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.
