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
On 2014/03/11 23:31:21, kpreid_google wrote:
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).
Done. I chose the order of the documentation on prepareExpr. While
there, I also noticed that "parseProgram" is now called
"parseFunctionBody", so I changed that too. PTAL in case I shouldn't be
disabling that here.
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
On 2014/03/11 23:31:21, kpreid_google wrote:
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.
I agree. Suggestions?
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.
