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

Posted to commits@shindig.apache.org by dd...@apache.org on 2012/01/26 17:12:08 UTC

svn commit: r1236265 - /shindig/trunk/features/src/main/javascript/features/container/service.js

Author: ddumont
Date: Thu Jan 26 16:12:08 2012
New Revision: 1236265

URL: http://svn.apache.org/viewvc?rev=1236265&view=rev
Log:
SHINDIG-1688 Container token refresh mechnism throws a fatal error after certain amount of time ( like 30 seconds ) -- FF only

Modified:
    shindig/trunk/features/src/main/javascript/features/container/service.js

Modified: shindig/trunk/features/src/main/javascript/features/container/service.js
URL: http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/container/service.js?rev=1236265&r1=1236264&r2=1236265&view=diff
==============================================================================
--- shindig/trunk/features/src/main/javascript/features/container/service.js (original)
+++ shindig/trunk/features/src/main/javascript/features/container/service.js Thu Jan 26 16:12:08 2012
@@ -432,7 +432,8 @@ osapi.container.Service.prototype.getCou
         containerTokenTTL = token ? (ttl * 1000 * 0.8) : containerTokenTTL;
         if (containerTokenTTL) {
           // Refresh again in 80% of the reported ttl
-          containerTimeout = setTimeout(gadgets.util.makeClosure(self, refresh), containerTokenTTL);
+          // Pass null in to closure because FF behaves un-expectedly when that param is not explicitly provided.
+          containerTimeout = setTimeout(gadgets.util.makeClosure(self, refresh, null), containerTokenTTL);
         }
 
         if (token) {