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
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... metaweta
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... metaweta
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... ihab . awad
- [Caja] Re: Modify Safe HTML Maker to optionally emit HTML as p... mikesamuel
