LGTM

http://codereview.appspot.com/2241042/diff/1/src/com/google/caja/plugin/PipelineMaker.java
File src/com/google/caja/plugin/PipelineMaker.java (right):

http://codereview.appspot.com/2241042/diff/1/src/com/google/caja/plugin/PipelineMaker.java#newcode236
src/com/google/caja/plugin/PipelineMaker.java:236: out.add(new
OptimizeJavascriptStage());
.given(JS).produces(CAJOLED_MODULE_UNOPTIMIZED)?

Else any client that requests CAJOLED_MODULE given JS always gets
optimized javascript.  Given that the only optimization currently
available is array optimization which produces code that is as
understandable as the original, its not critical.

http://codereview.appspot.com/2241042/

Reply via email to