Fix bug that Java2Script OO is not correctly run in Apple Safari 4.0 beta

zhourenjian · zhourenjian · commit 6e245bfaa817 · 2009-03-01T17:08:15.000Z
It is considered that WebKit engine over optimizes on function's arguments object. So we must make a clone copy of arguments before calling #apply
diff --git a/sources/net.sf.j2s.java.core/src/java/lang/Class.js b/sources/net.sf.j2s.java.core/src/java/lang/Class.js
@@ -1397,6 +1397,18 @@ Clazz.defineType = function (qClazzName, clazzFun, clazzParent, interfacez) {
 	return clazzFun;
 };
 
+Clazz.isSafari = (navigator.userAgent.indexOf ("Safari") != -1);
+Clazz.isSafari4Plus = false;
+if (Clazz.isSafari) {
+	var ua = navigator.userAgent;
+	var verIdx = ua.indexOf ("Version/");
+	if (verIdx  != -1) {
+		var verStr = ua.substring (verIdx + 8);
+		var verNumber = parseFloat (verStr);
+		Clazz.isSafari4Plus = verNumber >= 4.0;
+	}
+}
+
 /* protected */
 Clazz.instantialize = function (objThis, args) {
 	if (args != null && args.length == 1 && args[0] != null 
@@ -1416,6 +1428,13 @@ Clazz.instantialize = function (objThis, args) {
 			return this;
 		};
 	}
+	if (Clazz.isSafari4Plus) { // Fix bug of Safari 4.0+'s over-optimization
+		var argsClone = new Array ();
+		for (var k = 0; k < args.length; k++) {
+			argsClone[k] = args[k];
+		}
+		args = argsClone;
+	}
 	var c = objThis.construct;
 	if (c != null) {
 		if (objThis.con$truct == null) { // no need to init fields
