Fix names with '$' to not conflict with operators or internal helpers

tests/language/src/NamingTest.dart

 // Copyright (c) 2011, 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.
 
 class A {
   A() { NamingTest.count++; }
   foo(a, b) {
     Expect.equals(1, a);
     Expect.equals(2, b);
   }
@@ skipping 434 matching lines @@
     Expect.equals(10, b(10));
     Expect.equals(10, c(10));
   }
 
   static testPseudoTokens() {
     var EOS = 400;
     var ILLEGAL = 99;
     Expect.equals(499, EOS + ILLEGAL);
   }
 
+  static testDollar() {
+    Expect.equals(123, $(123).wrapped);
+    var x = new Object(), y = new Object();
+    Expect.identical(x, $(x).wrapped);
+    Expect.identical(y, $(x).$add(y));
+    Expect.identical(x, $(x).$negate());
+    Expect.equals(123, $(x) + x);
+    Expect.equals(444, -$(x));
+  }
+
   static void testMain() {
     count = 0;
     testExceptionNaming();
     testTmpNaming();
     testScopeNaming();
     testGlobalMangling();
     testMemberMangling();
     testFactoryMangling();
     testFunctionParameters();
     Bug4082360.test();
     Hoisting.test();
     testPseudoTokens();
+    testDollar();
   }
 }
 
+// Test that the generated JS names don't conflict with "$"
+class DartQuery {
+  Object wrapped;
+  DartQuery(this.wrapped);
+
+  $add(Object other) => other;
+  $negate() => wrapped;
+
+  operator +(Object other) => 123;
+  operator negate() => 444;
+}
+
+$add(Object first, Object second) => second;
+DartQuery $(Object obj) => new DartQuery(obj);
 
 // Ensure we don't have false positive. named constructor and methods
-// are in different namespaces, therefore it is ok to have a method 
+// are in different namespaces, therefore it is ok to have a method
 // called foo and a named constructor CLASS.foo
 class Naming1Test {
   Naming1Test.foo() { }
   foo() { }
 
   static void main(args) {
     var a = new Naming1Test.foo();
     a.foo();
   }
 }
 
 // Ensure we don't have false positive.
 class Naming2Test {
   Naming2Test() { }
   int get foo() { return 1; }
   set foo(x) { }
 
   static void main(args) {
     var a = new Naming2Test();
-    a.foo(2);
+    Expect.throws(() => a.foo(2), (e) => e is ObjectNotClosureException);
   }
 }
 
 // Ensure we don't have false positivesj.
 class Naming3Test {
   Naming3Test() { }
   operator negate() { }
-  negate() { }
+  negate() => 777;
 
   static void main(args) {
     var a = new Naming3Test();
-    a.negate(3);
+    Expect.equals(777, a.negate());
   }
 }
 
 main() {
   NamingTest.testMain();
+  Naming1Test.main(null);
+  Naming2Test.main(null);
+  Naming3Test.main(null);
 }