Re: Don't return in CajaContentRewriter's finally clause (issue2152046)

[Chirag Shah]

This change surfaces several (rather odd) unit test failures in
EndToEndTest. Do you see the same errors?

java.lang.AssertionError
        at com.google.caja.lexer.FilePosition.<init>(FilePosition.java:52)
        at 
com.google.caja.lexer.SourceBreaks.toFilePosition(SourceBreaks.java:73)
        at com.google.caja.lexer.FilePosition.span(FilePosition.java:137)
        at 
com.google.caja.parser.js.ObjProperty.<init>(ObjProperty.java:40)
        at 
com.google.caja.parser.js.ValueProperty.<init>(ValueProperty.java:17)

On Sun, Sep 12, 2010 at 10:24 PM,  <jas...@gmail.com> wrote:
> Absolutely LGTM.
>
> http://codereview.appspot.com/2152046/
>