Revert "Make LinkedHashMap also have a factory constructor and be customizable"

tests/corelib/map_test.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.
 
 library map_test;
 import "package:expect/expect.dart";
 import 'dart:collection';
 
 void main() {
   test(new HashMap());
@@ skipping 15 matching lines @@
   testWeirdStringKeys(new SplayTreeMap<String, String>());
 
   testNumericKeys(new Map());
   testNumericKeys(new Map<num, String>());
   testNumericKeys(new HashMap());
   testNumericKeys(new HashMap<num, String>());
   testNumericKeys(new HashMap(equals: identical));
   testNumericKeys(new HashMap<num, String>(equals: identical));
   testNumericKeys(new LinkedHashMap());
   testNumericKeys(new LinkedHashMap<num, String>());
-  testNumericKeys(new LinkedHashMap(equals: identical));
-  testNumericKeys(new LinkedHashMap<num, String>(equals: identical));
 
   testNaNKeys(new Map());
   testNaNKeys(new Map<num, String>());
   testNaNKeys(new HashMap());
   testNaNKeys(new HashMap<num, String>());
   testNaNKeys(new LinkedHashMap());
   testNaNKeys(new LinkedHashMap<num, String>());
   // Identity maps fail the NaN-keys tests because the test assumes that
   // NaN is not equal to NaN.
 
   testIdentityMap(new HashMap(equals: identical));
-  testIdentityMap(new LinkedHashMap(equals: identical));
 
-  testCustomMap(new HashMap(equals: myEquals, hashCode: myHashCode,
-                            isValidKey: (v) => v is Customer));
-  testCustomMap(new LinkedHashMap(equals: myEquals, hashCode: myHashCode,
-                                  isValidKey: (v) => v is Customer));
-  testCustomMap(new HashMap<Customer,dynamic>(equals: myEquals,
-                                              hashCode: myHashCode));
-
-  testCustomMap(new LinkedHashMap<Customer,dynamic>(equals: myEquals,
-                                                    hashCode: myHashCode));
+  testCustomMap(new HashMap(equals: myEquals, hashCode: myHashCode));
 
   testIterationOrder(new LinkedHashMap());
-  testIterationOrder(new LinkedHashMap(equals: identical));
-
-  testOtherKeys(new SplayTreeMap<int, int>());
-  testOtherKeys(new SplayTreeMap<int, int>((int a, int b) => a - b,
-                                           (v) => v is int));
-  testOtherKeys(new SplayTreeMap((int a, int b) => a - b,
-                                 (v) => v is int));
-  testOtherKeys(new HashMap<int, int>());
-  testOtherKeys(new HashMap<int, int>(equals: identical));
-  testOtherKeys(new HashMap<int, int>(hashCode: (v) => v.hashCode,
-                                      isValidKey: (v) => v is int));
-  testOtherKeys(new HashMap(equals: (int x, int y) => x == y,
-                            hashCode: (int v) => v.hashCode,
-                            isValidKey: (v) => v is int));
-  testOtherKeys(new LinkedHashMap<int, int>());
-  testOtherKeys(new LinkedHashMap<int, int>(equals: identical));
-  testOtherKeys(new LinkedHashMap<int, int>(hashCode: (v) => v.hashCode,
-                                       isValidKey: (v) => v is int));
-  testOtherKeys(new LinkedHashMap(equals: (int x, int y) => x == y,
-                                  hashCode: (int v) => v.hashCode,
-                                  isValidKey: (v) => v is int));
 }
 
 
 void test(Map map) {
   testDeletedElement(map);
   testMap(map, 1, 2, 3, 4, 5, 6, 7, 8);
   map.clear();
   testMap(map, "value1", "value2", "value3", "value4", "value5",
           "value6", "value7", "value8");
 }
@@ skipping 574 matching lines @@
   bool operator==(Object other) {
     if (other is! Customer) return false;
     Customer otherCustomer = other;
     return id == otherCustomer.id;
   }
 }
 
 int myHashCode(Customer c) => c.secondId;
 bool myEquals(Customer a, Customer b) => a.secondId == b.secondId;
 
-void testIterationOrder(Map map) {
+testIterationOrder(Map map) {
   var order = [0, 6, 4, 2, 7, 9, 7, 1, 2, 5, 3];
   for (int i = 0; i < order.length; i++) map[order[i]] = i;
   Expect.listEquals(map.keys.toList(), [0, 6, 4, 2, 7, 9, 1, 5, 3]);
   Expect.listEquals(map.values.toList(), [0, 1, 2, 8, 6, 5, 7, 9, 10]);
 }
-
-void testOtherKeys(Map<int, int> map) {
-  // Test that non-int keys are allowed in containsKey/remove/lookup.
-  // Custom hash sets and tree sets must be constructed so they don't
-  // use the equality/comparator on incompatible objects.
-
-  // This should not throw in either checked or unchecked mode.
-  map[0] = 0;
-  map[1] = 1;
-  map[2] = 2;
-  Expect.isFalse(map.containsKey("not an int"));
-  Expect.isFalse(map.containsKey(1.5));
-  Expect.isNull(map.remove("not an int"));
-  Expect.isNull(map.remove(1.5));
-  Expect.isNull(map["not an int"]);
-  Expect.isNull(map[1.5]);
-}