The domita.js changes look great, thanks! Only minor comments.


http://codereview.appspot.com/50041/diff/10037/8109
File src/com/google/caja/cajita-debugmode.js (left):

http://codereview.appspot.com/50041/diff/10037/8109#oldcode81
Line 81:
Why can we delete this stuff? Doesn't it do anything useful any more? Do
the debug mode tests work without it?

http://codereview.appspot.com/50041/diff/10037/8109#oldcode359
Line 359: ], 0);
Same question re deleting stuff.

http://codereview.appspot.com/50041/diff/10037/8107
File src/com/google/caja/plugin/domita.js (right):

http://codereview.appspot.com/50041/diff/10037/8107#newcode3243
Line 3243: // called by cajoled code, so we do not use
classUtils.inertClassCtor().
s/classUtils.inertClassCtor/inertCtor/

http://codereview.appspot.com/50041

Reply via email to