http://codereview.appspot.com/4535118/diff/44001/src/com/google/caja/plugin/domado.js File src/com/google/caja/plugin/domado.js (right):
http://codereview.appspot.com/4535118/diff/44001/src/com/google/caja/plugin/domado.js#newcode2386 src/com/google/caja/plugin/domado.js:2386: TameBackedNode.prototype.dispatchEvent = nodeMethod(function dispatchEvent(evt) { On 2011/07/21 19:54:46, MikeSamuel wrote:
No reasons. I may have added it during debugging so that it shows up
in
stack traces.
Removed. http://codereview.appspot.com/4535118/diff/48001/src/com/google/caja/plugin/domado.js File src/com/google/caja/plugin/domado.js (right): http://codereview.appspot.com/4535118/diff/48001/src/com/google/caja/plugin/domado.js#newcode912 src/com/google/caja/plugin/domado.js:912: var h = String(htmlFragment == null ? '' : htmlFragment); On 2011/07/21 19:54:46, MikeSamuel wrote:
Maybe add a comment that the == is intentional. Otherwise people are liable to "correct" it to ===.
Done. http://codereview.appspot.com/4535118/diff/48001/src/com/google/caja/plugin/domado.js#newcode922 src/com/google/caja/plugin/domado.js:922: : html.escapeAttrib(String(htmlFragment == null ? '' : htmlFragment)); On 2011/07/21 19:54:46, MikeSamuel wrote:
ditto.
Done. http://codereview.appspot.com/4535118/
