[Caja] Re: Modify Safe HTML Maker to optionally emit HTML as part of the JS.

[ihab . awad]

http://codereview.appspot.com/126062/diff/9001/10012
File src/com/google/caja/plugin/stages/CompileHtmlStage.java (right):

http://codereview.appspot.com/126062/diff/9001/10012#newcode127
Line 127: });
On 2009/10/27 22:02:35, MikeSamuel wrote:

There is a Concatenator(StringBuilder) ctor.

Done.

http://codereview.appspot.com/126062/diff/9001/10012#newcode129
Line 129: return new StringLiteral(
On 2009/10/27 22:02:35, MikeSamuel wrote:

Prefer StringLiteral.valueOf(Nodes.getFilePositionFor(node),

stringBuilder);

Done.

http://codereview.appspot.com/126062/diff/9001/10009
File tests/com/google/caja/util/TestUtil.java (right):

http://codereview.appspot.com/126062/diff/9001/10009#newcode19
Line 19: import com.google.caja.lexer.TokenConsumer;
On 2009/10/27 22:02:35, MikeSamuel wrote:

Not used

Done.

http://codereview.appspot.com/126062/diff/9001/10009#newcode30
Line 30: import com.google.caja.reporting.RenderContext;
On 2009/10/27 22:02:35, MikeSamuel wrote:

Not used

Done.

http://codereview.appspot.com/126062