dart2js: don't emit tear-off data if not needed.

pkg/compiler/lib/src/js_emitter/new_emitter/model_emitter.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library dart2js.new_js_emitter.model_emitter;
 
 import '../../dart2jslib.dart' show Compiler;
 import '../../dart_types.dart' show DartType;
 import '../../elements/elements.dart' show ClassElement;
 import '../../js/js.dart' as js;
@@ skipping 487 matching lines @@
   }
 
   /// JavaScript code template that implements parsing of a function descriptor.
   /// Descriptors are used in place of the actual JavaScript function
   /// definition in the output if additional information needs to be passed to
   /// facilitate the generation of tearOffs at runtime. The format is an array
   /// with the following fields:
   ///
   /// [Method.code]
   /// [DartMethod.callName]
+  /// [InstanceMethod.aliasName]
   /// [DartMethod.tearOffName]
   /// [JavaScriptBackend.isInterceptedMethod]
   /// functionType
-  /// [InstanceMethod.aliasName]
   ///
   /// followed by
   ///
   /// [ParameterStubMethod.name]
   /// [ParameterStubMethod.code]
   ///
   /// for each stub in [DartMethod.parameterStubs].
 
   static final String parseFunctionDescriptorBoilerplate = r"""
 function parseFunctionDescriptor(proto, name, descriptor) {
   if (descriptor instanceof Array) {
     proto[name] = descriptor[0];
     var funs = [descriptor[0]];
     funs[0].$callName = descriptor[1];
     for (var pos = 6; pos < descriptor.length; pos += 3) {
       var stub = descriptor[pos + 2];
       stub.$callName = descriptor[pos + 1];
       proto[descriptor[pos]] = stub;
       funs.push(stub);
     }
-    if (descriptor[2] != null) {
-      var isIntercepted = descriptor[3];
-      var reflectionInfo = descriptor[4];
-      proto[descriptor[2]] = 
+    if (descriptor[3] != null) {
+      var isIntercepted = descriptor[4];
+      var reflectionInfo = descriptor[5];
+      proto[descriptor[3]] =
           tearOff(funs, reflectionInfo, false, name, isIntercepted);
     }
     // Install the alias for super calls on the prototype chain.

[floitsch, 2015/02/09 12:41:41]

Move this code into right order.
First a check for desc[2]. Then for desc[3] and then starting at pos == 6.

[zarah, 2015/02/09 15:52:00]

Acknowledged.

-    if (descriptor[5] != null) {
-      proto[descriptor[5]] = descriptor[0];
+    if (descriptor[2] != null) {
+      proto[descriptor[2]] = descriptor[0];
     }
   } else {
     proto[name] = descriptor;
   }
 }
 """;
 
   js.Expression _generateFunctionType(DartType memberType) {
     if (memberType.containsTypeVariables) {
       js.Expression thisAccess = js.js(r'this.$receiver');
@@ skipping 13 matching lines @@
       js.Expression callName = stub.callName == null
           ? new js.LiteralNull()
           : js.string(stub.callName);
       return [js.string(stub.name), callName, stub.code];
     }
 
     if (method is InstanceMethod) {
       if (method.needsTearOff || method.aliasName != null) {
         /// See [parseFunctionDescriptorBoilerplate] for a full description of
         /// the format.
-        // [name, [function, callName, tearOffName, isIntercepted, functionType,
-        //     aliasName, stub1_name, stub1_callName, stub1_code, ...]
-        bool isIntercepted = backend.isInterceptedMethod(method.element);
+        // [name, [function, callName, aliasName, tearOffName, isIntercepted,
+        // functionType, stub1_name, stub1_callName, stub1_code, ...]
         var data = [method.code];
         data.add(js.string(method.callName));
-        data.add(js.string(method.tearOffName));
-        data.add(new js.LiteralBool(isIntercepted));
-        data.add(_generateFunctionType(method.type));
+
         if (method.aliasName != null) {
           data.add(js.string(method.aliasName));
         } else {
           data.add(new js.LiteralNull());
         }
+
+        if (method.needsTearOff) {

[floitsch, 2015/02/09 12:41:41]

You can only avoid adding these if there aren't any parameterStubs. Otherwise
your stubs are shifted and you miss some of them.

[zarah, 2015/02/09 15:52:00]

Acknowledged.

+          data.add(js.string(method.tearOffName));
+          bool isIntercepted = backend.isInterceptedMethod(method.element);
+          data.add(new js.LiteralBool(isIntercepted));
+          data.add(_generateFunctionType(method.type));
+        }
+
         data.addAll(method.parameterStubs.expand(makeNameCallNameCodeTriplet));
         return [js.string(method.name), new js.ArrayInitializer(data)];
       } else {
         // TODO(floitsch): not the most efficient way...
         return ([method]..addAll(method.parameterStubs))
             .expand(makeNameCodePair);
       }
     } else {
       return makeNameCodePair(method);
     }
@@ skipping 309 matching lines @@
 
   var end = Date.now();
   print('Setup: ' + (end - start) + ' ms.');
 
   #main();  // Start main.
 
 }(Date.now(), #code)
 }""";
 
 }