Reviewers: levik,

Message:
This takes the unchecked-in-parts of http://codereview.appspot.com/53052
and modifies/simplifies them further.  In particular, template library
scripts and styles will only be registered when the relevant library or
tag in the library is used.

Also, the NameTagHandler code (now unused) is deleted.

Missing functionality: template library injection for client-side
processing is too large (re-includes script and styles unnecessarily,
even if they were used already on the server).



Please review this at http://codereview.appspot.com/63167

Affected files:
M java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/TemplateRewriter.java A java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/DefaultTemplateLibrary.java D java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/NameTagHandler.java M java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/TemplateContext.java M java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/TemplateLibrary.java M java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/TemplateLibraryFactory.java A java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/TemplateResource.java A + java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/XmlTemplateLibrary.java M java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/TemplateRewriterTest.java M java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/DefaultTemplateProcessorTest.java M java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/RenderTagHandlerTest.java D java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/TemplateLibraryTest.java A + java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/XmlTemplateLibraryTest.java


Reply via email to