LGTM
https://codereview.appspot.com/12960044/diff/1002/src/com/google/caja/ses/repairES5.js
File src/com/google/caja/ses/repairES5.js (right):
https://codereview.appspot.com/12960044/diff/1002/src/com/google/caja/ses/repairES5.js#newcode3325
src/com/google/caja/ses/repairES5.js:3325: function
test_METHOD_IGNORES_SEALED() {
I don't like the use of uppercase here because it implies that
'METHOD_IGNORES_SEALED' is an actual problem ID. I suggest either
'test_method_IGNORES_SEALED' or keeping the original name. Ditto for the
other test and the repair.
I suggest inserting a comment above this line explaining what the
difference between the two test functions is.
https://codereview.appspot.com/12960044/
--
---
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.
