We must use the original element and not the default implementation element when checking if the co…

lib/compiler/implementation/ssa/builder.dart

Index: lib/compiler/implementation/ssa/builder.dart
===================================================================
--- lib/compiler/implementation/ssa/builder.dart	(revision 6728)
+++ lib/compiler/implementation/ssa/builder.dart	(working copy)
@@ -2156,7 +2156,7 @@
         ClassElement cls = element.enclosingElement;
         type = new HNonPrimitiveType(cls.type);
       } else if (element.isFactoryConstructor()

[floitsch, 2012/04/19 10:50:32]

Since we need to look at the original non-forwarded element anyways, let's only
look at that one.
Add a comment why.
And I don't think we shouldn't care if it is a constructor or a factory. In the
test-mockup it would be a factory, and in the actual implementation it would
probably be a constructor (since that's the kind in the interface).

[ngeoffray, 2012/04/19 16:17:52]

Done.

-                 && element === compiler.listClass) {
+                 && elements[node].enclosingElement === compiler.listClass) {
         type = HType.MUTABLE_ARRAY;
       }
       push(new HInvokeStatic(selector, inputs, type));