Some core changes required for the tools service

Fixed bug: myvn filetypes marks gifs as svn:mime-type=text/gif

Added a minimizing CSS printer.  This is svn cped from CssPrettyPrinter.

Got rid of some unnecessary parenthesization around unary operators.
We now render !!x as !!x instead of !(!x)

Updated version of google-code-prettify, and turned the second version
under doc/html into symlinks into third_party.

Allow constant folding to NaN which is rendered by RealLiteral as (0/0).

Fixed rendering of -0 to preserve sign.

Fixed some message strings that were confusing.

Added an immutable map constructor to Maps.
Previously we often wrote
  private static final Map<Foo, Bar> LOOKUP_TABLE;
  static {
    LOOKUP_TABLE = /* an empty mutable map */;
    LOOKUP_TABLE.put(...);
    // many more puts.
  }
Now you can write
  private static final Map<Foo, Bar> LOOKUP_TABLE
    = Maps.<Foo, Bar>immutableMap()
        .put(a, b)
        .put(c, d)
        .create();
So the result is immutable, and no need for static inializers or
temporary variables.

Added jakarta file upload jars and dependency to third_party.

Removed useless <permissions> from build.xml.  This was cruft from the
Rhino sandboxing CL.  It doesn't work in forked mode, and so we
modified the sandbox to never allow file access, and instead resolve
file: URLs using the class loader.


Submitted @3854

[MikeSamuel, 2009-11-17 21:54:58]

ping

[Jasvir, 2009-11-18 23:56:00]

LGTM

http://codereview.appspot.com/154063/diff/4001/4014
File src/com/google/caja/parser/js/Operation.java (right):

http://codereview.appspot.com/154063/diff/4001/4014#newcode398
src/com/google/caja/parser/js/Operation.java:398: if (op.getType() ==
OperatorType.PREFIX) { return false; }
Appears to directly contradict comment on Line 394.  In any case should be
before the above comment.

[MikeSamuel, 2009-11-19 00:37:09]

http://codereview.appspot.com/154063/diff/4001/4014
File src/com/google/caja/parser/js/Operation.java (right):

http://codereview.appspot.com/154063/diff/4001/4014#newcode398
src/com/google/caja/parser/js/Operation.java:398: if (op.getType() ==
OperatorType.PREFIX) { return false; }
On 2009/11/18 23:56:00, jasvir wrote:
> Appears to directly contradict comment on Line 394.  In any case should be
> before the above comment.

Fixed comment and moved line above.