Allow Object when doing lookups.

sdk/lib/_internal/compiler/implementation/lib/collection_patch.dart

Index: sdk/lib/_internal/compiler/implementation/lib/collection_patch.dart
diff --git a/sdk/lib/_internal/compiler/implementation/lib/collection_patch.dart b/sdk/lib/_internal/compiler/implementation/lib/collection_patch.dart
index 10afb9a49fb2fd2871071cdfe5673837ceb36c52..1db922fb2b408038ef8308f924bd3443a3309916 100644
--- a/sdk/lib/_internal/compiler/implementation/lib/collection_patch.dart
+++ b/sdk/lib/_internal/compiler/implementation/lib/collection_patch.dart
@@ -39,7 +39,7 @@ patch class HashMap<K, V> {
     return keys.map((each) => this[each]);
   }
 
-  patch bool containsKey(K key) {
+  patch bool containsKey(Object key) {
     if (_isStringKey(key)) {
       var strings = _strings;
       return (strings == null) ? false : _hasTableEntry(strings, key);

[Lasse Reichstein Nielsen, 2013/04/24 09:11:18]

It always pains me to see ?: expressions with a boolean constant branch.
This could be 
  return (strings != null) && _hasTableEntry(strings, key);
Likewise below.

@@ -54,7 +54,7 @@ patch class HashMap<K, V> {
     }
   }
 
-  patch bool containsValue(V value) {
+  patch bool containsValue(Object value) {
     return _computeKeys().any((each) => this[each] == value);
   }
 
@@ -312,7 +312,7 @@ class HashMapKeyIterable<E> extends IterableBase<E> {
     return new HashMapKeyIterator<E>(_map, _map._computeKeys());
   }
 
-  bool contains(E element) {
+  bool contains(Object element) {
     return _map.containsKey(element);
   }
 
@@ -394,7 +394,7 @@ patch class LinkedHashMap<K, V> {
     return keys.map((each) => this[each]);
   }
 
-  patch bool containsKey(K key) {
+  patch bool containsKey(Object key) {
     if (_isStringKey(key)) {
       var strings = _strings;
       if (strings == null) return false;
@@ -413,7 +413,7 @@ patch class LinkedHashMap<K, V> {
     }
   }
 
-  patch bool containsValue(V value) {
+  patch bool containsValue(Object value) {
     return keys.any((each) => this[each] == value);
   }
 
@@ -423,7 +423,7 @@ patch class LinkedHashMap<K, V> {
     });
   }
 
-  patch V operator[](K key) {
+  patch V operator[](Object key) {
     if (_isStringKey(key)) {
       var strings = _strings;
       if (strings == null) return null;
@@ -664,7 +664,7 @@ class LinkedHashMapKeyIterable<E> extends IterableBase<E> {
     return new LinkedHashMapKeyIterator<E>(_map, _map._modifications);
   }
 
-  bool contains(E element) {
+  bool contains(Object element) {
     return _map.containsKey(element);
   }