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() {
On 2013/08/15 16:43:11, kpreid2 wrote:
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.
Done.
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.
