You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by li...@apache.org on 2010/06/05 02:35:12 UTC

svn commit: r951625 - /shindig/trunk/features/src/main/javascript/features/container/init.js

Author: lindner
Date: Sat Jun  5 00:35:12 2010
New Revision: 951625

URL: http://svn.apache.org/viewvc?rev=951625&view=rev
Log:
SHINDIG-1355 | Patch from Michael Hermanto | Avoid double onload= callback by common container js

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

Modified: shindig/trunk/features/src/main/javascript/features/container/init.js
URL: http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/container/init.js?rev=951625&r1=951624&r2=951625&view=diff
==============================================================================
--- shindig/trunk/features/src/main/javascript/features/container/init.js (original)
+++ shindig/trunk/features/src/main/javascript/features/container/init.js Sat Jun  5 00:35:12 2010
@@ -48,23 +48,6 @@
     }
   }
 
-  /**
-   * Call a callback function if specified in &onload= query param. This is
-   * required for dynamic source script inclusion, which is asynchronous.
-   */
-  function runOnloadCallback() {
-    var scriptSrc = getLastScriptSrc();
-    if (scriptSrc) {
-      var onload = shindig.container.util.getParamValue(scriptSrc, 'onload');
-      if (onload) {
-        var re = /(^[A-Za-z0-9_]+$)/;
-        if (re.test(onload) && (typeof window[onload] === "function")) {
-          window[onload]();
-        }
-      }
-    }
-  }
-
   function getLastScriptSrc() {
     var scriptEls = document.getElementsByTagName('script');
     return (scriptEls.length > 0)
@@ -74,5 +57,4 @@
 
   initializeGadgetsConfig();
   initializeGlobalVars();
-  runOnloadCallback();
 })();