lima golf tango mike


http://codereview.appspot.com/88157/diff/1/3
File src/com/google/caja/plugin/domita.js (right):

http://codereview.appspot.com/88157/diff/1/3#newcode1341
Line 1341: throw new Error(NOT_EDITABLE);
Ah! Good catch. Thanks for adding this.

http://codereview.appspot.com/88157/diff/1/3#newcode1359
Line 1359: TameBackedNode.prototype.replaceChild = function (newChild,
oldChild) {
eep, we had the parameters named bass-ackwards-wise. Thanks for catching
that too!

http://codereview.appspot.com/88157/diff/1/3#newcode1576
Line 1576: TamePseudoNode.prototype.replaceChild = function () {
The original had the wrong signature (1 instead of 2 args); I would
think it would be better to fix the signature, as a form of
documentation for code readers, rather than delete it, even if the
arguments are never used.

http://codereview.appspot.com/88157

Reply via email to