Store the aliasName of a method in the model.

pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart

 // Copyright (c) 2013, 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;
 
 /// This class should morph into something that makes it easy to build
 /// JavaScript representations of libraries, class-sides, and instance-sides.
 /// Initially, it is just a placeholder for code that is moved from
 /// [CodeEmitterTask].
@@ skipping 234 matching lines @@
   void addMemberMethod(DartMethod method, ClassBuilder builder) {
     final FunctionElement member = method.element;
     String name = method.name;
     final FunctionSignature parameters = member.functionSignature;
     jsAst.Expression code = method.code;
     final bool needsStubs = method.needsStubs;
     final bool canBeApplied = method.canBeApplied;
     bool canTearOff = method.needsTearOff;
     String tearOffName = method.tearOffName;
     final bool isClosure = method is InstanceMethod && method.isClosure;
-    final bool hasSuperAlias = method is InstanceMethod && method.hasSuperAlias;
+    final String superAlias = method is InstanceMethod ? method.aliasName

[floitsch, 2015/01/29 15:24:35]

Remove the `final` if that makes it fit on one line.

These 'final's don't seem to add any value...

[herhut, 2015/02/03 10:20:16]

Removed them all.

+                                                       : null;
+    final bool hasSuperAlias = superAlias != null;
 
     bool canBeReflected = backend.isAccessibleByReflection(member);
     // During incremental compilation, we have to assume that reflection
     // *might* get enabled.
     if (compiler.hasIncrementalSupport) canBeReflected = true;
 
     if (canBeReflected) {
       // Patch the effects of reflection into the received information.
       bool isNotApplyTarget = member.isConstructor ||
                               member.isAccessor ||
@@ skipping 49 matching lines @@
     // ...
     // P.   Unmangled name (if reflectable).
     // P+1. First metadata (if reflectable).
     // ...
     // TODO(ahe): Consider one of the parameter counts can be replaced by the
     // length property of the JavaScript function object.
 
     List<jsAst.Expression> expressions = <jsAst.Expression>[];
 
     // Create the optional aliasing entry if this method is called via super.
-    if (backend.isAliasedSuperMember(member)) {
-      expressions.add(new jsAst.LiteralString(
-          '"${namer.getNameOfAliasedSuperMember(member)}"'));
+    if (hasSuperAlias) {
+      expressions.add(new jsAst.LiteralString('"${superAlias}"'));
     }
 
     expressions.add(code);
 
     final bool onlyNeedsSuperAlias =
         !(canTearOff || canBeReflected || canBeApplied || needsStubs);
 
     if (onlyNeedsSuperAlias) {
       jsAst.ArrayInitializer arrayInit =
             new jsAst.ArrayInitializer(expressions);
@@ skipping 113 matching lines @@
     jsAst.ArrayInitializer arrayInit =
       new jsAst.ArrayInitializer(expressions.toList());
     compiler.dumpInfoTask.registerElementAst(member,
         builder.addProperty(name, arrayInit));
   }
 
   void addMemberField(Field field, ClassBuilder builder) {
     // For now, do nothing.
   }
 }