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#newcode117
Line 117: "html", renderDomAsJsStringLiteral(node)));
LGTM
http://codereview.appspot.com/126062/diff/9001/10012#newcode127
Line 127: });
There is a Concatenator(StringBuilder) ctor.
http://codereview.appspot.com/126062/diff/9001/10012#newcode128
Line 128: Nodes.render(node, new
RenderContext(tc).withEmbeddable(true));
OK
http://codereview.appspot.com/126062/diff/9001/10012#newcode129
Line 129: return new StringLiteral(
Prefer StringLiteral.valueOf(Nodes.getFilePositionFor(node),
stringBuilder);
http://codereview.appspot.com/126062/diff/9001/10003
File tests/com/google/caja/plugin/stages/CompileHtmlStageTest.java
(right):
http://codereview.appspot.com/126062/diff/9001/10003#newcode25
Line 25: public final class CompileHtmlStageTest extends
PipelineStageTestCase {
Ok, this was the result of an {svn cp} so the testcases to the left were
not deleted.
http://codereview.appspot.com/126062/diff/9001/10003#newcode31
Line 31: Job.JobType.JAVASCRIPT));
Short. Sweet. Spiffy.
http://codereview.appspot.com/126062/diff/9001/10003#newcode41
Line 41: Job.JobType.JAVASCRIPT),
cool
http://codereview.appspot.com/126062/diff/9001/10003#newcode44
Line 44: + " throw 'Translated code must never be executed';\n"
Ok, this is the paranoid guff at the top of TranslatedCode.render()
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;
Not used
http://codereview.appspot.com/126062/diff/9001/10009#newcode30
Line 30: import com.google.caja.reporting.RenderContext;
Not used
http://codereview.appspot.com/126062
