You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by ga...@apache.org on 2011/03/03 07:43:13 UTC

svn commit: r1076536 - in /shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite: CssResponseRewriter.java StyleTagProxyEmbeddedUrlsRewriter.java

Author: gagan
Date: Thu Mar  3 06:43:12 2011
New Revision: 1076536

URL: http://svn.apache.org/viewvc?rev=1076536&view=rev
Log:
Patch by anupama.dutta | Issue 4250051: Changing access for some CssResponseRewriter and StyleTagProxyEmbeddedUrlsRewriter variables | http://codereview.appspot.com/4250051/

Modified:
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/CssResponseRewriter.java
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/StyleTagProxyEmbeddedUrlsRewriter.java

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/CssResponseRewriter.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/CssResponseRewriter.java?rev=1076536&r1=1076535&r2=1076536&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/CssResponseRewriter.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/CssResponseRewriter.java Thu Mar  3 06:43:12 2011
@@ -61,8 +61,8 @@ public class CssResponseRewriter impleme
   private static final Logger LOG = Logger.getLogger(classname,MessageKeys.MESSAGES);
 
   private final CajaCssParser cssParser;
-  private final ProxyUriManager proxyUriManager;
-  private final ContentRewriterFeature.Factory rewriterFeatureFactory;
+  protected final ProxyUriManager proxyUriManager;
+  protected final ContentRewriterFeature.Factory rewriterFeatureFactory;
 
   @Inject
   public CssResponseRewriter(CajaCssParser cssParser,
@@ -222,11 +222,11 @@ public class CssResponseRewriter impleme
     return new UriMaker(wrapped, config);
   }
 
-  public static final class UriMaker {
-    private final ProxyUriManager wrapped;
-    private final ContentRewriterFeature.Config config;
+  public static class UriMaker {
+    protected final ProxyUriManager wrapped;
+    protected final ContentRewriterFeature.Config config;
 
-    private UriMaker(ProxyUriManager wrapped, ContentRewriterFeature.Config config) {
+    public UriMaker(ProxyUriManager wrapped, ContentRewriterFeature.Config config) {
       this.wrapped = wrapped;
       this.config = config;
     }

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/StyleTagProxyEmbeddedUrlsRewriter.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/StyleTagProxyEmbeddedUrlsRewriter.java?rev=1076536&r1=1076535&r2=1076536&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/StyleTagProxyEmbeddedUrlsRewriter.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/StyleTagProxyEmbeddedUrlsRewriter.java Thu Mar  3 06:43:12 2011
@@ -34,9 +34,9 @@ import java.util.List;
  * @since 2.0.0
  */
 public class StyleTagProxyEmbeddedUrlsRewriter extends DomWalker.Rewriter {
-  private final ContentRewriterFeature.Config config;
-  private final ProxyUriManager proxyUriManager;
-  private final CssResponseRewriter cssRewriter;
+  protected final ContentRewriterFeature.Config config;
+  protected final ProxyUriManager proxyUriManager;
+  protected final CssResponseRewriter cssRewriter;
 
   @Inject
   public StyleTagProxyEmbeddedUrlsRewriter(ContentRewriterFeature.DefaultConfig config,