LGTM
https://codereview.appspot.com/67720043/diff/1/src/com/google/caja/plugin/html-emitter.js
File src/com/google/caja/plugin/html-emitter.js (right):
https://codereview.appspot.com/67720043/diff/1/src/com/google/caja/plugin/html-emitter.js#newcode593
src/com/google/caja/plugin/html-emitter.js:593: rewriteFunctionCalls:
false
Please write the properties either in alphabetical order or the order in
which their documentation lists them, or other consistent and
non-arbitrary order.
Also, consistent whitespace (no space before colon in any line).
https://codereview.appspot.com/67720043/diff/1/src/com/google/caja/ses/startSES.js
File src/com/google/caja/ses/startSES.js (right):
https://codereview.appspot.com/67720043/diff/1/src/com/google/caja/ses/startSES.js#newcode784
src/com/google/caja/ses/startSES.js:784: // coordination with the list
of mitigation options in
Insofar as SES should be able to hypothetically exist as a separate
library which Caja depends on, it would be nice if such coordination
were somehow not necessary.
https://codereview.appspot.com/67720043/
--
---
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/d/optout.
