Throw when reflecting on elements not covered by a `MirrorsUsed` annotation.

tests/compiler/dart2js_extra/mirror_invalid_field_access2_test.dart

Index: tests/compiler/dart2js_extra/mirror_invalid_field_access2_test.dart
diff --git a/tests/compiler/dart2js_extra/mirror_invalid_field_access2_test.dart b/tests/compiler/dart2js_extra/mirror_invalid_field_access2_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..e972bbb25b6a97457ffaf309008aea4f84a72b5e
--- /dev/null
+++ b/tests/compiler/dart2js_extra/mirror_invalid_field_access2_test.dart
@@ -0,0 +1,33 @@
+// 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.
+
+// Test that we cannot reflect on elements not covered by the `MirrorsUsed`
+// annotation.
+
+library test;
+
+@MirrorsUsed(targets: 'C.foo')

[ahe, 2013/08/26 20:41:15]

What happens if you define a class D and add it to targets:

class D {
  var foo;
  var bar;
}

In this case, I think the test will fail because mangledNames will contain an
entry for D.bar, which will be mistaken for the one for C.bar.

+import 'dart:mirrors';
+
+import 'package:expect/expect.dart';
+
+class C {
+  var foo;
+  var bar;
+}
+
+main() {
+  var c = new C();
+
+  c.bar = 1;
+  var local = c.bar;
+
+  var mirror = reflect(c);
+  Expect.equals(1, mirror.setField(const Symbol('foo'), 1).reflectee);
+  Expect.equals(1, mirror.getField(const Symbol('foo')).reflectee);
+  Expect.throws(() => mirror.setField(const Symbol('bar'),  2),
+                (e) => e is NoSuchMethodError);
+  Expect.throws(() => mirror.getField(const Symbol('bar')),
+                (e) => e is NoSuchMethodError);
+}