Split MirrorsData methods and MirrorsDataImpl data by element kind.

pkg/compiler/lib/src/util/emptyset.dart

Index: pkg/compiler/lib/src/util/emptyset.dart
diff --git a/pkg/compiler/lib/src/util/emptyset.dart b/pkg/compiler/lib/src/util/emptyset.dart
index 26474da95ef7aa49965397723f8f6c112a6d5815..809df0b624d018c5df2c3d0f5b28b17957932cd0 100644
--- a/pkg/compiler/lib/src/util/emptyset.dart
+++ b/pkg/compiler/lib/src/util/emptyset.dart
@@ -18,17 +18,17 @@ class ImmutableEmptySet<E> extends IterableBase<E> implements Set<E> {
   bool add(E element) => _immutableError;
   void addAll(Iterable<E> elements) => _immutableError;
 
-  E lookup(E element) => null;
-  bool remove(E element) => false;
-  void removeAll(Iterable<E> elements) {}
+  E lookup(Object element) => null;

[Emily Fortuna, 2017/04/04 18:27:13]

why this change? If you want Object everywhere, there are a few other places you
can replace E with Object like the return value on this line...

[Johnni Winther, 2017/04/05 06:59:59]

This class was made when `Iterable` used `E` in [lookup], [remove], [removeAll],
[retainAll], [contains] and [containsAll]. `Iterable` has since been changed to
use `Object`, this change just follows this.

+  bool remove(Object element) => false;
+  void removeAll(Iterable<Object> elements) {}
   void removeWhere(bool test(E element)) {}
-  void retainAll(Iterable<E> elements) {}
+  void retainAll(Iterable<Object> elements) {}
   void retainWhere(bool test(E element)) {}
   void forEach(void action(E element)) {}
   void clear() {}
 
-  bool contains(E element) => false;
-  bool containsAll(Iterable<E> other) => other.isEmpty;
+  bool contains(Object element) => false;
+  bool containsAll(Iterable<Object> other) => other.isEmpty;
 
   Set<E> union(Set<E> other) => new Set.from(other);
   Set<E> intersection(Set<E> other) => this;