Fix for Issue 2878

frog/member.dart

 // Copyright (c) 2012, 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.
 
 /** A formal parameter to a [Method]. */
 class Parameter {
   FormalNode definition;
   Member method;
 
   String name;
@@ skipping 69 matching lines @@
   bool get hasDefaultValue() =>
     definition.value.span.start != definition.span.start;
 }
 
 
 class Member extends Element {
   final Type declaringType;
 
   Member genericMember;
 
+  // A root string for getter and setter names.  This is used e.g. to ensure
+  // that fields with the same Dart name but different jsnames (due to native
+  // name directives) still have a common getter name.  Is null when there is no
+  // renaming.
+  String _jsnameRoot;
+
   Member(String name, Type declaringType)
       : this.declaringType = declaringType,
         super(name, declaringType);
 
   String mangleJsName() {
     var mangled = super.mangleJsName();
     if (declaringType != null && declaringType.isTop) {
       return JsNames.getValid(mangled);
     } else {
       // We don't need to mangle native member names unless
@@ skipping 33 matching lines @@
   bool _provideSetter = false;
 
   bool get isNative() => false;
   String get constructorName() {
     world.internalError('cannot be a constructor', span);
   }
 
   void provideGetter() {}
   void provideSetter() {}
 
+  String get jsnameOfGetter() => 'get\$$jsnameRoot';
+  String get jsnameOfSetter() => 'set\$$jsnameRoot';
+  String get jsnameRoot() => _jsnameRoot != null ? _jsnameRoot : _jsname;
+
   Member get initDelegate() {
     world.internalError('cannot have initializers', span);
   }
   void set initDelegate(ctor) {
     world.internalError('cannot have initializers', span);
   }
 
   Value computeValue() {
     world.internalError('cannot have value', span);
   }
@@ skipping 720 matching lines @@
     _provideOptionalParamInfo = true;
     if (isStatic) {
       // ensure the type is generated.
       // TODO(sigmund): can we avoid generating the entire type, but only what
       // we need?
       declaringType.markUsed();
       var type = declaringType.isTop ? '' : '${declaringType.jsname}.';
       return new Value(functionType, '$type$jsname', node.span);
     }
     _provideGetter = true;
-    return new Value(functionType, '${target.code}.get\$$jsname()', node.span);
+    return new Value(functionType, '${target.code}.$jsnameOfGetter()', node.span);
   }
 
   /**
    * Checks if the named arguments are in their natural or 'home' positions,
    * i.e. they may be passed directly without inserting, deleting or moving the
    * arguments to correspond with the parameters.
    */
   bool namesInHomePositions(Arguments args) {
     if (!args.hasNames) return true;
 
@@ skipping 531 matching lines @@
         f(member);
       });
     });
   }
 
   bool isEmpty() {
     return factories.getValues()
         .every((Map constructors) => constructors.isEmpty());
   }
 }