Make enum values array canonical

Make enum values array canonical

BUG=24594
R=rmacnak@google.com

Committed: https://github.com/dart-lang/sdk/commit/647ada25ed6f6ec745fd5e5885cbeb53c3741b1c

[hausner, 2015-10-15 21:53:30]

hausner@google.com changed reviewers:
+ rmacnak@google.com

[rmacnak, 2015-10-15 22:20:21]

https://codereview.chromium.org/1407103002/diff/20001/tests/language/enum_tes...
File tests/language/enum_test.dart (right):

https://codereview.chromium.org/1407103002/diff/20001/tests/language/enum_tes...
tests/language/enum_test.dart:25: Expect.identical(const [Enum2.A],
Enum2.values);
(sigh, generics) Specs says this should be const <Enum2>[Enum2.A].

[rmacnak, 2015-10-15 22:34:31]

On 2015/10/15 22:20:21, rmacnak wrote:
>
https://codereview.chromium.org/1407103002/diff/20001/tests/language/enum_tes...
> File tests/language/enum_test.dart (right):
> 
>
https://codereview.chromium.org/1407103002/diff/20001/tests/language/enum_tes...
> tests/language/enum_test.dart:25: Expect.identical(const [Enum2.A],
> Enum2.values);
> (sigh, generics) Specs says this should be const <Enum2>[Enum2.A].

LGTM. dart2js seems to do this do.

[hausner, 2015-10-15 22:53:01]

Thanks

https://codereview.chromium.org/1407103002/diff/20001/tests/language/enum_tes...
File tests/language/enum_test.dart (right):

https://codereview.chromium.org/1407103002/diff/20001/tests/language/enum_tes...
tests/language/enum_test.dart:25: Expect.identical(const [Enum2.A],
Enum2.values);
On 2015/10/15 22:20:21, rmacnak wrote:
> (sigh, generics) Specs says this should be const <Enum2>[Enum2.A].

As discussed offline, I'll leave the type argument for a later change.

[hausner, 2015-10-15 22:56:38]

Committed patchset #2 (id:20001) manually as
647ada25ed6f6ec745fd5e5885cbeb53c3741b1c (presubmit successful).