lgtm++
https://codereview.appspot.com/13271044/diff/1/src/com/google/caja/plugin/domado.js
File src/com/google/caja/plugin/domado.js (right):
https://codereview.appspot.com/13271044/diff/1/src/com/google/caja/plugin/domado.js#newcode1445
src/com/google/caja/plugin/domado.js:1445: // call, syntax occurs occur
async.
You mean "syntax errors". And yes, I noticed. Very nice.
https://codereview.appspot.com/13271044/diff/1/tests/com/google/caja/plugin/test-domado-events-guest.html
File tests/com/google/caja/plugin/test-domado-events-guest.html (right):
https://codereview.appspot.com/13271044/diff/1/tests/com/google/caja/plugin/test-domado-events-guest.html#newcode509
tests/com/google/caja/plugin/test-domado-events-guest.html:509: ['str',
'toString'].forEach(function(name) {
why are "str" and "toString" special names to test here?
https://codereview.appspot.com/13271044/
--
---
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.
