First pass at a javascript auto-documentation tool

First pass at a javascript auto-documentation tool that handles use of
closure for information hiding properly.

See first src/com/google/caja/tools/jsdoc/package.html which sketches
out the overall design and how various parts fit together.


Submitted @3826

[MikeSamuel, 2009-01-09 19:50:26]

ping

[Jasvir, 2009-09-11 18:06:07]

http://codereview.appspot.com/8706/diff/44649/44654
File tests/com/google/caja/tools/jsdoc/JsdocMainTest.java (right):

http://codereview.appspot.com/8706/diff/44649/44654#newcode1
Line 1: // Copyright (C) 2008 Google Inc.
2009

http://codereview.appspot.com/8706/diff/53720/53726
File tests/com/google/caja/tools/jsdoc/UpdocParserTest.java (right):

http://codereview.appspot.com/8706/diff/53720/53726#newcode29
Line 29: + "$ Math.abs(-5)\n"
What in the rhino setup prevents js-java bridge from being used to access and
modify the local filesystem?

[MikeSamuel, 2009-09-11 18:49:48]

http://codereview.appspot.com/8706/diff/44649/44654
File tests/com/google/caja/tools/jsdoc/JsdocMainTest.java (right):

http://codereview.appspot.com/8706/diff/44649/44654#newcode1
Line 1: // Copyright (C) 2008 Google Inc.
On 2009/09/11 18:06:07, jasvir wrote:
> 2009

Not when I wrote it.

http://codereview.appspot.com/8706/diff/53720/53726
File tests/com/google/caja/tools/jsdoc/UpdocParserTest.java (right):

http://codereview.appspot.com/8706/diff/53720/53726#newcode29
Line 29: + "$ Math.abs(-5)\n"
On 2009/09/11 18:06:07, jasvir wrote:
> What in the rhino setup prevents js-java bridge from being used to access and
> modify the local filesystem?

Nothing yet.
This is actually using the JS math builtin.
The java version would be accessed via java.lang.Math.abs.

[MikeSamuel, 2009-09-11 19:47:13]

http://codereview.appspot.com/8706/diff/53720/53726
File tests/com/google/caja/tools/jsdoc/UpdocParserTest.java (right):

http://codereview.appspot.com/8706/diff/53720/53726#newcode29
Line 29: + "$ Math.abs(-5)\n"
On 2009/09/11 18:49:49, MikeSamuel wrote:
> On 2009/09/11 18:06:07, jasvir wrote:
> > What in the rhino setup prevents js-java bridge from being used to access
and
> > modify the local filesystem?
> 
> Nothing yet.
> This is actually using the JS math builtin.
> The java version would be accessed via java.lang.Math.abs.

I do have a TODO to sandbox the rhino implementation so that it can't reach the
FS or network.
I know of a project that does this, and can ask the guys involved for the magic
incantations to do this.

[Jasvir, 2009-10-23 04:48:11]

I had a look over the non-ancillary changes - all of which are minor.  LGTM

http://codereview.appspot.com/8706/diff/61601/61614
File src/com/google/caja/plugin/stages/CompileHtmlStage.java (right):

http://codereview.appspot.com/8706/diff/61601/61614#newcode90
Line 90: boolean onlyJs = false;
This looks like staging for another set of changes.  Probably best to remove it
from this CL or provide some means where onlyJS gets passed in by the pipeline.

[MikeSamuel, 2009-10-23 05:00:13]

http://codereview.appspot.com/8706/diff/61601/61614
File src/com/google/caja/plugin/stages/CompileHtmlStage.java (right):

http://codereview.appspot.com/8706/diff/61601/61614#newcode90
Line 90: boolean onlyJs = false;
This was actually me just sanity-checking something before responding to Ihab's
code review.  I'd meant to delete it, but never did.

[MikeSamuel, 2009-10-23 19:13:06]

snapshotted