Hide synthetic metadata field exposed in r35926.

runtime/lib/mirrors.cc

Index: runtime/lib/mirrors.cc
diff --git a/runtime/lib/mirrors.cc b/runtime/lib/mirrors.cc
index f39effa54a4645e52178c9939bde1f8652428a52..5eb63fdcaf6023717ffc2bb7c75476a927245e15 100644
--- a/runtime/lib/mirrors.cc
+++ b/runtime/lib/mirrors.cc
@@ -1057,8 +1057,10 @@ DEFINE_NATIVE_ENTRY(ClassMirror_members, 2) {
   Field& field = Field::Handle();
   for (intptr_t i = 0; i < num_fields; i++) {
     field ^= fields.At(i);
-    member_mirror = CreateVariableMirror(field, owner_mirror);
-    member_mirrors.Add(member_mirror);
+    if (!field.is_synthetic()) {

[Ivan Posva, 2014/07/01 14:27:48]

Do we need this extra field in the Field or can't we just base it on the name?

[rmacnak, 2014/07/01 20:08:20]

Will consider this in a follow up CL. FWIW this does not increase the size of a
Field.

+      member_mirror = CreateVariableMirror(field, owner_mirror);
+      member_mirrors.Add(member_mirror);
+    }
   }
 
   Function& func = Function::Handle();
@@ -1145,8 +1147,10 @@ DEFINE_NATIVE_ENTRY(LibraryMirror_members, 2) {
       }
     } else if (entry.IsField()) {
       const Field& field = Field::Cast(entry);
-      member_mirror = CreateVariableMirror(field, owner_mirror);
-      member_mirrors.Add(member_mirror);
+      if (!field.is_synthetic()) {
+        member_mirror = CreateVariableMirror(field, owner_mirror);
+        member_mirrors.Add(member_mirror);
+      }
     } else if (entry.IsFunction()) {
       const Function& func = Function::Cast(entry);
       if (func.kind() == RawFunction::kRegularFunction ||