http://codereview.appspot.com/126062/diff/4001/3088
File src/com/google/caja/plugin/html-emitter.js (right):

http://codereview.appspot.com/126062/diff/4001/3088#newcode75
Line 75: base.innerHTML = htmlString;
As discussed, I'm fine with clobbering, but maybe we could fail fast if
there is content.
Something like
  // TODO: we could append the HTML but no clients need that yet.
  if (base.firstChild) { throw new Error(...); }

http://codereview.appspot.com/126062

Reply via email to