Use normal equality for const maps.

tests/language/const_map_test.dart

 // Copyright (c) 2014, 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.
 
 import "package:expect/expect.dart";
 
-// Regression test for Issue 16407.
-
-void main() {
-  foo(null, true);
-  foo('x', false);
+confuse(x) {

[Lasse Reichstein Nielsen, 2014/03/03 10:31:24]

Document "confuse": Used to prevent static optimizations and inlining by naking
the value too hard to predict at compile time.

[floitsch, 2014/03/03 10:42:38]

Done.

+  if (new DateTime.now() == 42) return confuse(2);
+  return x;
 }
 
-var foo = (x, result) {
-  Expect.equals(result, x is Null, '$x is Null');
-};
+main() {
+  // Make sure that const maps use the == operator and not identical.

[Lasse Reichstein Nielsen, 2014/03/03 10:31:24]

Way that the spec doesn't require this explicitly, but it does so implicitly (by
allowing int and String keys even if they override operator==).

Luckily the VM agrees.

(And, also luckliy, you can't use doubles as keys, or we would have to figure
out what const{1:42,1.0:37} would mean :).

[floitsch, 2014/03/03 10:42:38]

Added string test and added more comment.

+  var m = const { 1: 42 };
+  Expect.equals(42, m[confuse(1.0)]);
+}