Refactor 'precajole' targets

Refactor 'precajole' targets to sequester testing "light" precajole map
from production "heavy" map. This avoids problems with previous setup
wherein the "light" contents were being written out to the production
"precajole.jar" output.

[felix8a, 2012-02-16 23:33:58]

lgtm++

http://codereview.appspot.com/5677069/diff/3002/build.xml
File build.xml (right):

http://codereview.appspot.com/5677069/diff/3002/build.xml#newcode506
build.xml:506: <target name="pluginc"
depends="PluginCompiler,tools,precajole-light"
this messes up some of my manual testing habits, but I'll adapt.

(the bin/cajole script only needs 'ant jars' built, which is fast because it
only builds pluginc not the playground.  so I'll have to remember to do 'ant
precajole jars')

http://codereview.appspot.com/5677069/diff/3002/build.xml#newcode1830
build.xml:1830: <precajole spec="${precajolesrc}/heavy.xml"
dest="${precajolelib}" />
please either add a <precajole> of light.xml here, or include light.xml in
heavy.xml.  it's helpful to have the light js files in the actual deployed
playground.

[ihab.awad, 2012-02-17 19:24:34]

http://codereview.appspot.com/5677069/diff/3002/build.xml
File build.xml (right):

http://codereview.appspot.com/5677069/diff/3002/build.xml#newcode506
build.xml:506: <target name="pluginc"
depends="PluginCompiler,tools,precajole-light"
On 2012/02/16 23:33:58, felix8a wrote:
> the bin/cajole script only needs 'ant jars' built, which is fast because it
> only builds pluginc not the playground.

I understand this part.

> so I'll have to remember to do 'ant precajole jars'

It's not clear to me how this follows from the first. What am I missing?

http://codereview.appspot.com/5677069/diff/3002/build.xml#newcode1830
build.xml:1830: <precajole spec="${precajolesrc}/heavy.xml"
dest="${precajolelib}" />
On 2012/02/16 23:33:58, felix8a wrote:
> please either add a <precajole> of light.xml here, or include light.xml in
> heavy.xml.  it's helpful to have the light js files in the actual deployed
> playground.

Ok, I'll figure out something. While it may be desirable to have the light.xml
stuff in the Playground, people who build Caja for deployment somewhere should
be able to have a precajole.jar that does not contain testing entries.

[felix8a, 2012-02-23 11:01:44]

lgtm

http://codereview.appspot.com/5677069/diff/10001/tests/com/google/caja/plugin...
File tests/com/google/caja/plugin/GeneralBrowserTest.java (right):

http://codereview.appspot.com/5677069/diff/10001/tests/com/google/caja/plugin...
tests/com/google/caja/plugin/GeneralBrowserTest.java:30: /*
commented out for testing?

[ihab.awad, 2012-02-23 16:25:31]

On Thu, Feb 23, 2012 at 3:01 AM, <felix8a@gmail.com> wrote:

> commented out for testing?
>

Yes, my bad, fixing. :)

Ihab

-- 
Ihab A.B. Awad, Palo Alto, CA

[ihab.awad, 2012-02-23 18:33:40]

@4785