You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by jo...@apache.org on 2011/08/05 02:46:18 UTC
svn commit: r1154054 -
/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/js/DeferJsProcessor.java
Author: johnh
Date: Fri Aug 5 00:46:17 2011
New Revision: 1154054
URL: http://svn.apache.org/viewvc?rev=1154054&view=rev
Log:
No need to clearJs() in defer processor.
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/js/DeferJsProcessor.java
Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/js/DeferJsProcessor.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/js/DeferJsProcessor.java?rev=1154054&r1=1154053&r2=1154054&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/js/DeferJsProcessor.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/js/DeferJsProcessor.java Fri Aug 5 00:46:17 2011
@@ -62,7 +62,6 @@ public class DeferJsProcessor extends Ba
// TODO: Instead of clearing, do a replacement of feature impl with defer stubs.
// Clearing has an effect of ignoring previous processors work.
if (needDefers) {
- builder.clearJs();
builder.appendAllJs(getSurfaceJsContents(featureRegistry, FEATURE_NAME));
}
builder.appendAllJs(resp.build());