dart:js - speed up constructor calls with few arguments.

sdk/lib/js/dart2js/js_dart2js.dart

Index: sdk/lib/js/dart2js/js_dart2js.dart
diff --git a/sdk/lib/js/dart2js/js_dart2js.dart b/sdk/lib/js/dart2js/js_dart2js.dart
index 6de50320ae1f5770d1a63cd21d13b48a083f50aa..aedddafe18fb624e87f60c5c1a63de1d69fc8aa0 100644
--- a/sdk/lib/js/dart2js/js_dart2js.dart
+++ b/sdk/lib/js/dart2js/js_dart2js.dart
@@ -141,6 +141,38 @@ class JsObject {
     if (arguments == null) {
       return _wrapToDart(JS('', 'new #()', constr));
     }
+
+    if (JS('bool', '# instanceof Array', arguments)) {
+      int argumentCount = JS('int', '#.length', arguments);
+      switch (argumentCount) {
+        case 0:
+          return _wrapToDart(JS('', 'new #()', constr));
+
+        case 1:
+          var arg0 = _convertToJS(JS('', '#[0]', arguments));
+          return _wrapToDart(JS('', 'new #(#)', constr, arg0));
+
+        case 2:
+          var arg0 = _convertToJS(JS('', '#[0]', arguments));
+          var arg1 = _convertToJS(JS('', '#[1]', arguments));
+          return _wrapToDart(JS('', 'new #(#, #)', constr, arg0, arg1));

[Siggi Cherem (dart-lang), 2015/05/26 16:35:18]

any reason why not write these 3 lines as:
return _wrapToDArt(JS('',
    'new #(#[0], #[1])',
    constr, arguments, arguments));

or use a named template 'new #ctor(#argumets[0], #arguments[1])' ?

[sra1, 2015/05/26 17:58:58]

Not really, but the expression gets big and any automatic layout will either
make it ugly or make it take a lot of lines.
I like this but named template parameters are not available to the JS 'function'
since the arguments are decomposed/spread at compile time.

+
+        case 3:
+          var arg0 = _convertToJS(JS('', '#[0]', arguments));
+          var arg1 = _convertToJS(JS('', '#[1]', arguments));
+          var arg2 = _convertToJS(JS('', '#[2]', arguments));
+          return _wrapToDart(JS('', 'new #(#, #, #)', constr, arg0, arg1, arg2));

[Siggi Cherem (dart-lang), 2015/05/26 16:35:18]

wrap

[sra1, 2015/05/26 17:58:59]

Done.

+
+        case 4:
+          var arg0 = _convertToJS(JS('', '#[0]', arguments));
+          var arg1 = _convertToJS(JS('', '#[1]', arguments));
+          var arg2 = _convertToJS(JS('', '#[2]', arguments));
+          var arg3 = _convertToJS(JS('', '#[3]', arguments));
+          return _wrapToDart(
+              JS('', 'new #(#, #, #, #)', constr, arg0, arg1, arg2, arg3));
+      }
+    }
+
     // The following code solves the problem of invoking a JavaScript
     // constructor with an unknown number arguments.
     // First bind the constructor to the argument list using bind.apply().
@@ -154,9 +186,16 @@ class JsObject {
     JS('String', 'String(#)', factoryFunction);
     // This could return an UnknownJavaScriptObject, or a native
     // object for which there is an interceptor
-    var jsObj = JS('JavaScriptObject', 'new #()', factoryFunction);

[Siggi Cherem (dart-lang), 2015/05/26 16:35:18]

I'm not familiar with this annotation here - any reason to remove it at this
point? Or is it equivalent to not provide anything in the first arg?

[sra1, 2015/05/26 17:58:58]

The constructor could be a constructor for something understood by Dart, e.g
Blob or typed array.

+    var jsObj = JS('', 'new #()', factoryFunction);
 
     return _wrapToDart(jsObj);
+
+    // TODO(sra): Investigate:
+    //
+    //     var jsObj = JS('', 'Object.create(#.prototype)', constr);
+    //     JS('', '#.apply(#, #)', constr, jsObj,
+    //         []..addAll(arguments.map(_convertToJS)));
+    //     return _wrapToDart(jsObj);
   }
 
   /**
@@ -510,25 +549,28 @@ dynamic _convertToJS(dynamic o) {
   // `undefined` in Javascprit). See dartbug.com/20305 for details.
   if (o == null || o is String || o is num || o is bool) {
     return o;
-  } else if (o is Blob || o is Event || o is KeyRange || o is ImageData
-      || o is Node || o is TypedData || o is Window) {
+  }
+  if (o is JsObject) {

[Siggi Cherem (dart-lang), 2015/05/26 16:35:18]

just to make sure I'm not missing something, these changes in this function are
just style, right?

[sra1, 2015/05/26 17:58:59]

Yes.  Initially I had just moved || to the end of the line.

+    return o._jsObject;
+  }
+  if (o is Blob || o is Event || o is KeyRange || o is ImageData || o is Node ||
+      o is TypedData || o is Window) {
     return o;
-  } else if (o is DateTime) {
+  }
+  if (o is DateTime) {
     return Primitives.lazyAsJsDate(o);
-  } else if (o is JsObject) {
-    return o._jsObject;
-  } else if (o is Function) {
+  }
+  if (o is Function) {
     return _getJsProxy(o, _JS_FUNCTION_PROPERTY_NAME, (o) {
       var jsFunction = _convertDartFunction(o);
       // set a property on the JS closure referencing the Dart closure
       _defineProperty(jsFunction, _DART_CLOSURE_PROPERTY_NAME, o);
       return jsFunction;
     });
-  } else {
-    var ctor = _dartProxyCtor;
-    return _getJsProxy(o, _JS_OBJECT_PROPERTY_NAME,
-        (o) => JS('', 'new #(#)', ctor, o));
   }
+  var ctor = _dartProxyCtor;
+  return _getJsProxy(o, _JS_OBJECT_PROPERTY_NAME,
+      (o) => JS('', 'new #(#)', ctor, o));
 }
 
 Object _getJsProxy(o, String propertyName, createProxy(o)) {
@@ -567,13 +609,13 @@ JsObject _wrapToDart(o) {
   if (JS('bool', 'typeof # == "function"', o)) {
     return _getDartProxy(o, _DART_CLOSURE_PROPERTY_NAME,
         (o) => new JsFunction._fromJs(o));
-  } else if (JS('bool', '# instanceof Array', o)) {
+  }
+  if (JS('bool', '# instanceof Array', o)) {
     return _getDartProxy(o, _DART_OBJECT_PROPERTY_NAME,
         (o) => new JsArray._fromJs(o));
-  } else {
-    return _getDartProxy(o, _DART_OBJECT_PROPERTY_NAME,
-        (o) => new JsObject._fromJs(o));
   }
+  return _getDartProxy(o, _DART_OBJECT_PROPERTY_NAME,
+      (o) => new JsObject._fromJs(o));
 }
 
 Object _getDartProxy(o, String propertyName, createProxy(o)) {