remove canonicalType

remove canonicalType

it is unused now since the extension type system replaced it.

there's a potentially interesting use case supported, e.g. ES6 subclasses of JS Array, but we can ignore that for now (IMO)

R=sra@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/256de552f91ca50f9c0bfb9315687341be26ecf8

[Jennifer Messerly, 2016-05-11 17:51:05]

Description was changed from

==========
remove canonicalType
it is unused
==========

to

==========
remove canonicalType

it is unused now since the extension type system replaced it.

there's a potentially interesting use case supported, e.g. ES6 subclasses of JS
Array, but we can ignore that for now (IMO)
==========

[Jennifer Messerly, 2016-05-11 17:51:05]

jmesserly@google.com changed reviewers:
+ leafp@google.com

[Jennifer Messerly, 2016-05-11 17:51:21]

jmesserly@google.com changed reviewers:
+ sra@google.com

[Jennifer Messerly, 2016-05-11 17:51:21]

As we were just talking about :)

[sra1, 2016-05-11 17:53:40]

lgtm

[Jennifer Messerly, 2016-05-11 17:57:02]

Description was changed from

==========
remove canonicalType

it is unused now since the extension type system replaced it.

there's a potentially interesting use case supported, e.g. ES6 subclasses of JS
Array, but we can ignore that for now (IMO)
==========

to

==========
remove canonicalType

it is unused now since the extension type system replaced it.

there's a potentially interesting use case supported, e.g. ES6 subclasses of JS
Array, but we can ignore that for now (IMO)

R=sra@google.com

Committed:
https://github.com/dart-lang/dev_compiler/commit/256de552f91ca50f9c0bfb931568...
==========

[Jennifer Messerly, 2016-05-11 17:57:03]

Committed patchset #1 (id:1) manually as
256de552f91ca50f9c0bfb9315687341be26ecf8 (presubmit successful).