Check whether type parameters are loaded before emitting.

This fixes #190.

BUG=
[email protected]

Review URL: https://codereview.chromium.org/1143253003

Author: leafpetersen

pkg/dev_compiler/lib/src/codegen/js_codegen.dart

@@ -1000,6 +1000,9 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
           type.normalParameterTypes.every(_typeIsLoaded));
     }
     if (type.isDynamic || type.isVoid || type.isBottom) return true;
+    if (type is ParameterizedType && !type.typeArguments.every(_typeIsLoaded)) {
+      return false;
+    }
     return _loader.isLoaded(type.element);
   }
 

pkg/dev_compiler/test/codegen/expect/functions.js

@@ -0,0 +1,19 @@
+var functions = dart.defineLibrary(functions, {});
+var core = dart.import(core);
+(function(exports, core) {
+  'use strict';
+  function bootstrap() {
+    return dart.setType([new Foo()], core.List$(Foo));
+  }
+  dart.fn(bootstrap, () => dart.functionType(core.List$(Foo), []));
+  class Foo extends core.Object {}
+  dart.setSignature(Foo, {});
+  function main() {
+    core.print(bootstrap()[core.$get](0));
+  }
+  dart.fn(main, dart.void, []);
+  // Exports:
+  exports.bootstrap = bootstrap;
+  exports.Foo = Foo;
+  exports.main = main;
+})(functions, core);

pkg/dev_compiler/test/codegen/expect/functions.txt

@@ -0,0 +1 @@
+// Messages from compiling functions.dart

pkg/dev_compiler/test/codegen/functions.dart

@@ -0,0 +1,14 @@
+List<Foo> bootstrap() {
+  return <Foo>[new Foo()];
+}
+
+typedef B A2B<A, B>(A x);
+
+A2B<Foo, Foo> id(A2B<Foo, Foo> f) => f;
+
+class Foo {
+}
+
+void main() {
+  print(bootstrap()[0]);
+}