You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@shindig.apache.org by ch...@apache.org on 2010/10/15 19:46:31 UTC

svn commit: r1023038 - /shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java

Author: chirag
Date: Fri Oct 15 17:46:30 2010
New Revision: 1023038

URL: http://svn.apache.org/viewvc?rev=1023038&view=rev
Log:
Only parse the lockedDomainSuffix when locked domain is enabled
Code Review: http://codereview.appspot.com/2524041/

Modified:
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java?rev=1023038&r1=1023037&r2=1023038&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java Fri Oct 15 17:46:30 2010
@@ -74,8 +74,10 @@ public class DefaultIframeUriManager imp
 
     Collection<String> containers = config.getContainers();
     List<String> ldSuffixes = Lists.newArrayListWithCapacity(containers.size());
-    for (String container : containers) {
-      ldSuffixes.add(getReqVal(container, LOCKED_DOMAIN_SUFFIX_KEY));
+    if (ldEnabled) {
+      for (String container : containers) {
+        ldSuffixes.add(getReqVal(container, LOCKED_DOMAIN_SUFFIX_KEY));
+      }
     }
     this.ldSuffixes = Collections.unmodifiableList(ldSuffixes);
   }