Fix class patching involving type parameters: type parameters of patched classes

Fix class patching involving type parameters: type parameters of patched classes
were not finalized.
Simplify and fix function type alias finalization (remove dummy alias owner).
Make sure no unfinalized types or unresolved classes are written to a snapshot.
Verify that all pending classes are finalized before writing a snapshot.
Make snapshot_test more resilient to finalization errors.
Add missing import to snapshot_test.dart.
Fix typo in growable_array.dart.
Committed: https://code.google.com/p/dart/source/detail?r=17962

[siva, 2013-02-01 00:31:57]

lgtm

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/object.cc
File runtime/vm/object.cc (right):

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/object.cc#newc...
runtime/vm/object.cc:1921: Class& result = Class::Handle(New<Instance>(name,
script, token_pos));
const Class& result?

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/parser.cc#newc...
runtime/vm/parser.cc:3402: Class& function_type_alias = Class::Handle(
const Class& function_type......

[regis, 2013-02-01 02:03:44]

Thanks!

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/object.cc
File runtime/vm/object.cc (right):

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/object.cc#newc...
runtime/vm/object.cc:1921: Class& result = Class::Handle(New<Instance>(name,
script, token_pos));
On 2013/02/01 00:31:57, siva wrote:
> const Class& result?

Done.

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

https://codereview.chromium.org/12123002/diff/10001/runtime/vm/parser.cc#newc...
runtime/vm/parser.cc:3402: Class& function_type_alias = Class::Handle(
On 2013/02/01 00:31:57, siva wrote:
> const Class& function_type......

Done.