You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by aw...@apache.org on 2007/07/11 02:32:55 UTC

svn commit: r555124 - /myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js

Author: awiner
Date: Tue Jul 10 17:32:54 2007
New Revision: 555124

URL: http://svn.apache.org/viewvc?view=rev&rev=555124
Log:
Add a FIXME for some non-portable JS

Modified:
    myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js

Modified: myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js?view=diff&rev=555124&r1=555123&r2=555124
==============================================================================
--- myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js (original)
+++ myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js Tue Jul 10 17:32:54 2007
@@ -1149,6 +1149,9 @@
 
     if (blurFunc)
     {
+      // FIXME: this is *NOT* portable.  Safari, for example,
+      // does not implement toString() on functions this way,
+      // and nothing requires that it does
       var valFunc = blurFunc.toString();
 
       // whack off the beginning and end of the function, leaving the content