Support user generated custom native JS classes.

pkg/compiler/lib/src/inferrer/type_graph_nodes.dart

Index: pkg/compiler/lib/src/inferrer/type_graph_nodes.dart
diff --git a/pkg/compiler/lib/src/inferrer/type_graph_nodes.dart b/pkg/compiler/lib/src/inferrer/type_graph_nodes.dart
index c1d51cc2f797f663aa49234eebffef938b083be1..cfbb8b32c7582345687dec504cd4c91349ba5cac 100644
--- a/pkg/compiler/lib/src/inferrer/type_graph_nodes.dart
+++ b/pkg/compiler/lib/src/inferrer/type_graph_nodes.dart
@@ -404,7 +404,7 @@ class MemberTypeInformation extends ElementTypeInformation
       giveUp(inferrer);
       return safeType(inferrer);
     }
-    if (inferrer.isNativeElement(element)) {
+    if (inferrer.isNativeElement(element) && !element.isConstructor) {

[Siggi Cherem (dart-lang), 2015/10/06 22:38:01]

is this change still needed?

If so, maybe we need to change it a bit to make it more clear that this is about
js-interop?

[Jacob, 2015/10/13 01:19:23]

removed the .isConstructor check here and added
|| element.isConstructor
to the assert bellow.
Not really sure if this is right and don't have enough context on this code to
really judge it. Let me know if this seems reasonable or if we should handle
this some other way.
My guess is that we simply never had to worry about constructors that were
native elements.

       // Use the type annotation as the type for native elements. We
       // also give up on inferring to make sure this element never
       // goes in the work queue.