Check whether type parameters are loaded before emitting

Check whether type parameters are loaded before emitting.

This fixes #190.

BUG=
R=vsm@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/0c7e8a86912c8ee8946ae8b8202ce00b0015b931

[Leaf, 2015-05-20 16:27:48]

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

[vsm, 2015-05-20 16:33:34]

lgtm with comment

https://codereview.chromium.org/1143253003/diff/1/lib/src/codegen/js_codegen....
File lib/src/codegen/js_codegen.dart (right):

https://codereview.chromium.org/1143253003/diff/1/lib/src/codegen/js_codegen....
lib/src/codegen/js_codegen.dart:996: if (type is InterfaceType &&
!type.typeArguments.every(_typeIsLoaded)) {
Should this be ParameterizedType instead of InterfaceType?  typeArguments is
defined in that base type, and that'll cover parameterized function types.

[Leaf, 2015-05-20 16:43:24]

https://codereview.chromium.org/1143253003/diff/1/lib/src/codegen/js_codegen....
File lib/src/codegen/js_codegen.dart (right):

https://codereview.chromium.org/1143253003/diff/1/lib/src/codegen/js_codegen....
lib/src/codegen/js_codegen.dart:996: if (type is InterfaceType &&
!type.typeArguments.every(_typeIsLoaded)) {
On 2015/05/20 16:33:34, vsm wrote:
> Should this be ParameterizedType instead of InterfaceType?  typeArguments is
> defined in that base type, and that'll cover parameterized function types.

Yes, good catch.  Fixed now, test added.

[vsm, 2015-05-20 16:46:08]

lgtm

[Leaf, 2015-05-20 16:47:44]

Committed patchset #2 (id:20001) manually as
0c7e8a86912c8ee8946ae8b8202ce00b0015b931 (presubmit successful).