Remove URI resolution from TemplateCompiler

Modify TemplateCompiler so it no longer resolves URI-valued HTML
attributes itself; instead, it relies solely on the PluginEnvironment
to do all resolution.

[felix8a, 2009-07-10 22:52:01]

lgtm

http://codereview.appspot.com/88152/diff/1/2
File tests/com/google/caja/plugin/templates/TemplateCompilerTest.java (right):

http://codereview.appspot.com/88152/diff/1/2#newcode310
Line 310: final Holder<ExternalReference> savedRef = new
Holder<ExternalReference>();
why not just
 ExternalReference savedRef;
?

[ihab.awad, 2009-07-11 00:12:59]

http://codereview.appspot.com/88152/diff/1/2
File tests/com/google/caja/plugin/templates/TemplateCompilerTest.java (right):

http://codereview.appspot.com/88152/diff/1/2#newcode310
Line 310: final Holder<ExternalReference> savedRef = new
Holder<ExternalReference>();
On 2009/07/10 22:52:01, felix8a wrote:
> why not just
>  ExternalReference savedRef;
> ?

It has to be final in order to be accessible from the anonymous inner class that
follows.

[ihab.awad, 2009-07-11 00:13:43]

@3565

[felix8a, 2009-07-11 00:17:18]

http://codereview.appspot.com/88152/diff/1/2
File tests/com/google/caja/plugin/templates/TemplateCompilerTest.java (right):

http://codereview.appspot.com/88152/diff/1/2#newcode310
Line 310: final Holder<ExternalReference> savedRef = new
Holder<ExternalReference>();
On 2009/07/11 00:12:59, ihab.awad wrote:
> On 2009/07/10 22:52:01, felix8a wrote:
> > why not just
> >  ExternalReference savedRef;
> > ?
> 
> It has to be final in order to be accessible from the anonymous inner class
that
> follows.

ah, yeah. ok.