Revert "dart2js: --fast-startup: use alias for 'this'"

pkg/compiler/lib/src/js_emitter/startup_emitter/fragment_emitter.dart

 // Copyright (c) 2015, 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.
 
 part of dart2js.js_emitter.startup_emitter.model_emitter;
 
 /// The name of the property that stores the tear-off getter on a static
 /// function.
 ///
 /// This property is only used when isolates are used.
@@ skipping 635 matching lines @@
   ///
   /// The constructor is statically built.
   js.Expression emitConstructor(Class cls) {
     js.Name name = cls.name;
     // If the class is not directly instantiated we only need it for inheritance
     // or RTI. In either case we don't need its fields.
     if (cls.isNative || !cls.isDirectlyInstantiated) {
       return js.js('function #() { }', name);
     }
 
-    var statements = <js.Statement>[];
-    var parameters = <js.Name>[];
-    var thisRef;
-
-    // If there are many references to `this`, cache it in a local.
-    if (cls.fields.length + (cls.hasRtiField ? 1 : 0) >= 4) {
-      statements.add(js.js.statement('var _ = this;'));
-      thisRef = js.js('_');
-    } else {
-      thisRef = js.js('this');
+    List<js.Name> fieldNames =
+        cls.fields.map((Field field) => field.name).toList();
+    if (cls.hasRtiField) {
+      fieldNames.add(namer.rtiFieldJsName);
     }
 
-    for (Field field in cls.fields) {
-      js.Name paramName = field.name;
-      parameters.add(paramName);
-      statements
-          .add(js.js.statement('#.# = #', [thisRef, field.name, paramName]));
-    }
+    Iterable<js.Name> assignments = fieldNames.map((js.Name field) {
+      return js.js("this.#field = #field", {"field": field});
+    });
 
-    if (cls.hasRtiField) {
-      js.Name paramName = namer.rtiFieldJsName;
-      parameters.add(paramName);
-      statements.add(js.js
-          .statement('#.# = #', [thisRef, namer.rtiFieldJsName, paramName]));
-    }
+    // TODO(sra): Cache 'this' in a one-character local for 4 or more uses of
+    // 'this'. i.e. "var _=this;_.a=a;_.b=b;..."
 
-    return js.js('function #(#) { # }', [name, parameters, statements]);
+    // TODO(sra): Separate field and field initializer parameter names so the
+    // latter may be fully minified.
+
+    return js.js('function #(#) { # }', [name, fieldNames, assignments]);
   }
 
   /// Emits the prototype-section of the fragment.
   ///
   /// This section updates the prototype-property of all constructors in the
   /// global holders.
   js.Statement emitPrototypes(Fragment fragment) {
     List<js.Statement> assignments = fragment.libraries
         .expand((Library library) => library.classes)
         .map((Class cls) {
@@ skipping 751 matching lines @@
     }
     statements.add(js.js.statement("setOrUpdateInterceptorsByTag(#);",
         js.objectLiteral(interceptorsByTag)));
     statements.add(
         js.js.statement("setOrUpdateLeafTags(#);", js.objectLiteral(leafTags)));
     statements.addAll(subclassAssignments);
 
     return wrapPhase('nativeSupport', new js.Block(statements));
   }
 }