Rename cajoling service to proxy; remove dead code.

This is a very rough cleanup; the main thing it doesn't do is refactor
the individual methods in ProxyServlet to remove unnecessary code.

* Combine CajolingServlet, CajolingService, AbstractCajolingHandler, and
  ProxyHandler into single-purpose class ProxyServlet; remove code
  relating to supporting multiple handlers.
* Rename entry point URL from '/cajole' to '/proxy' and remove unused
  URL parameters relating to cajoling.
* Remove unrelated dead code CssPrettyPrinter, CssMinimalPrinter.

@r5649

[felix8a, 2013-12-18 19:43:23]

lgtm

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
File src/com/google/caja/service/ProxyServlet.java (right):

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
src/com/google/caja/service/ProxyServlet.java:70: // TODO(kpreid): This code
needs cleanup; it has many relics of its origin
most of my potential comments are basically this, so it doesn't seem necessary
to elaborate further

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
src/com/google/caja/service/ProxyServlet.java:96: * @deprecated Do not use this;
instead pass in the correct host argument
there are three other constructors, none of them take just a host argument.

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
src/com/google/caja/service/ProxyServlet.java:369: // regarding what constitutes
a JS identifier.
this implementation seems pretty bizarre to me. maybe add a todo to rethink
this?

https://codereview.appspot.com/43650043/diff/1/tests/com/google/caja/service/...
File tests/com/google/caja/service/ProxyServletTest.java (right):

https://codereview.appspot.com/43650043/diff/1/tests/com/google/caja/service/...
tests/com/google/caja/service/ProxyServletTest.java:72: public void
testCheckIdentifier() throws Exception {
final

[kpreid2, 2013-12-18 20:33:06]

This is a very rough cleanup; the main thing it doesn't do is refactor
the individual methods in ProxyServlet to remove unnecessary code.

* Combine CajolingServlet, CajolingService, AbstractCajolingHandler, and
  ProxyHandler into single-purpose class ProxyServlet; remove code
  relating to supporting multiple handlers.
* Rename entry point URL from '/cajole' to '/proxy' and remove unused
  URL parameters relating to cajoling.
* Remove unrelated dead code: CssPrettyPrinter, CssMinimalPrinter,
  ModuleManager, and a couple of BuildInfo subclasses.

[kpreid2, 2013-12-18 20:34:36]

New snapshot.

As well as changes noted below, I deleted some more dead code, namely
ModuleManager and some subclasses of BuildInfo. (BuildInfo is now only used for
the playground's version display, I think, and can go away when we replace the
playground.)

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
File src/com/google/caja/service/ProxyServlet.java (right):

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
src/com/google/caja/service/ProxyServlet.java:96: * @deprecated Do not use this;
instead pass in the correct host argument
On 2013/12/18 19:43:24, felix8a wrote:
> there are three other constructors, none of them take just a host argument.

It seems that nothing except the UriFetcher is actually used any more, so
deleted that stuff. The zero-argument ctor is no longer marked deprecated.

https://codereview.appspot.com/43650043/diff/1/src/com/google/caja/service/Pr...
src/com/google/caja/service/ProxyServlet.java:369: // regarding what constitutes
a JS identifier.
On 2013/12/18 19:43:24, felix8a wrote:
> this implementation seems pretty bizarre to me. maybe add a todo to rethink
> this?

Done.

https://codereview.appspot.com/43650043/diff/1/tests/com/google/caja/service/...
File tests/com/google/caja/service/ProxyServletTest.java (right):

https://codereview.appspot.com/43650043/diff/1/tests/com/google/caja/service/...
tests/com/google/caja/service/ProxyServletTest.java:72: public void
testCheckIdentifier() throws Exception {
On 2013/12/18 19:43:24, felix8a wrote:
> final

Done.

[felix8a, 2013-12-19 15:50:08]

lgtm