Remove recently introduced FunctionType vm class by merging it into class Type.

Remove recently introduced FunctionType vm class by merging it into class Type.
Fix syntax used when printing function type with optional named parameters.
Fix several issues discovered with cloning function types.

R=hausner@google.com

Committed: https://github.com/dart-lang/sdk/commit/966aea003bd319f9577caa19756563365236a952

[regis, 2016-03-18 21:58:23]

regis@google.com changed reviewers:
+ hausner@google.com, rmacnak@google.com, srdjan@google.com

[hausner, 2016-03-21 20:59:18]

lgtm

LGTM

https://codereview.chromium.org/1815733003/diff/1/runtime/vm/object.cc
File runtime/vm/object.cc (left):

https://codereview.chromium.org/1815733003/diff/1/runtime/vm/object.cc#oldcod...
runtime/vm/object.cc:15985: 
Nit: keep this empty line.

[regis, 2016-03-21 21:07:09]

Thanks!

https://codereview.chromium.org/1815733003/diff/1/runtime/vm/object.cc
File runtime/vm/object.cc (left):

https://codereview.chromium.org/1815733003/diff/1/runtime/vm/object.cc#oldcod...
runtime/vm/object.cc:15985: 
On 2016/03/21 20:59:17, hausner wrote:
> Nit: keep this empty line.

Done.

[regis, 2016-03-21 21:09:00]

Description was changed from

==========
Remove recently introduced FunctionType vm class by merging it into class Type.
Fix syntax used when printing function type with optional named parameters.
Fix several issues discovered with cloning function types.
==========

to

==========
Remove recently introduced FunctionType vm class by merging it into class Type.
Fix syntax used when printing function type with optional named parameters.
Fix several issues discovered with cloning function types.

R=hausner@google.com

Committed:
https://github.com/dart-lang/sdk/commit/966aea003bd319f9577caa19756563365236a952
==========

[regis, 2016-03-21 21:09:02]

Committed patchset #2 (id:20001) manually as
966aea003bd319f9577caa19756563365236a952 (presubmit successful).

[srdjan, 2016-03-21 21:39:30]

lgtm