Ensure locations are always provided, but don't store them yet.

pkg/front_end/lib/src/fasta/dill/dill_library_builder.dart

Index: pkg/front_end/lib/src/fasta/dill/dill_library_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/dill/dill_library_builder.dart b/pkg/front_end/lib/src/fasta/dill/dill_library_builder.dart
index 3041dfd70e0cd77d9421e9039f31b0385d3f0bb0..4e544ad79db8421b182d754b550255cf8676a7b3 100644
--- a/pkg/front_end/lib/src/fasta/dill/dill_library_builder.dart
+++ b/pkg/front_end/lib/src/fasta/dill/dill_library_builder.dart
@@ -49,7 +49,8 @@ class DillLibraryBuilder extends LibraryBuilder<KernelTypeBuilder, Library> {
 
   Library library;
 
-  DillLibraryBuilder(this.uri, this.loader);
+  DillLibraryBuilder(Uri uri, this.loader)
+      : uri = uri, super(uri);

[Johnni Winther, 2017/02/13 08:42:03]

Isn't initializing formal also parameters now by default?

[ahe, 2017/02/13 10:44:23]

I don't know. I'd like  to be able to continue performance testing this on a
Dart VM from November.

 
   get scope => internalError("Scope not supported");
 
@@ -112,7 +113,7 @@ class DillLibraryBuilder extends LibraryBuilder<KernelTypeBuilder, Library> {
   }
 
   KernelInvalidTypeBuilder buildAmbiguousBuilder(
-      String name, Builder builder, Builder other) {
-    return new KernelInvalidTypeBuilder(name, this);
+      String name, Builder builder, Builder other, int charOffset) {
+    return new KernelInvalidTypeBuilder(name, charOffset, fileUri);
   }
 }