Replace _superApply with _super for performance critical hooks

mar10 · mar10 · commit f9013e985f5f · 2017-02-25T08:57:14.000+01:00
Close mar10#684
diff --git a/doc/fancytree_benchmarks.txt b/doc/fancytree_benchmarks.txt
@@ -0,0 +1,13 @@
+# Fancytree benchmarks
+
+Results for 
+    http://localhost:8080/test/unit/test-bench.html
+
+## MacBook Pro early 2011, i5, 2,3GHz, 4GB
+   Safari 10.0.3
+ - 2017-02-14: jQuery 3.1.1, jQuery UI 1.12.1, 
+   use _super instead of _superApply:                                    2220 ms
+                                                                         1440
+ - 2017-02-14: jQuery 3.1.1, jQuery UI 1.12.1:                           2180 ms
+                                                                         1413
+ - 2017-02-14: jQuery 1.12.1, jQuery UI 1.11.4:                          2440 ms
diff --git a/src/jquery.fancytree.table.js b/src/jquery.fancytree.table.js
@@ -336,7 +336,7 @@ $.ui.fancytree.registerExtension({
 			opts = ctx.options,
 			isStatusNode = node.isStatusNode();
 
-		res = this._superApply(arguments);
+		res = this._super(ctx, title);
 
 		if( node.isRootNode() ) {
 			return res;
@@ -364,7 +364,7 @@ $.ui.fancytree.registerExtension({
 			node = ctx.node,
 			opts = ctx.options;
 
-		this._superApply(arguments);
+		this._super(ctx);
 
 		$(node.tr).removeClass("fancytree-node");
 		// indent
@@ -436,6 +436,7 @@ $.ui.fancytree.registerExtension({
 	treeDestroy: function(ctx) {
 		this.$container.find("tbody").empty();
 		this.$source && this.$source.removeClass("ui-helper-hidden");
+		return this._superApply(arguments);
 	}
 	/*,
 	treeSetFocus: function(ctx, flag) {
diff --git a/test/unit/test-bench.html b/test/unit/test-bench.html
@@ -2,10 +2,10 @@
 <html>
 <head>
 	<title>Fancytree benchmarks</title>
-	<script src="//code.jquery.com/jquery-1.12.1.min.js"></script>
-	<script src="//code.jquery.com/ui/1.11.4/jquery-ui.min.js"></script>
+	<script src="//code.jquery.com/jquery-3.1.1.min.js"></script>
+	<script src="//code.jquery.com/ui/1.12.1/jquery-ui.min.js"></script>
 
-	<link rel="stylesheet" href="../../lib/qunit.css" media="screen" />
+	<link href="../../lib/qunit.css" rel="stylesheet" media="screen" />
 	<script src="../../lib/qunit.js"></script>
 
 	<link href="../../src/skin-win8/ui.fancytree.css" rel="stylesheet">
