Reapply CL: first step for modular support in fasta

pkg/front_end/lib/src/fasta/kernel/verifier.dart

Index: pkg/front_end/lib/src/fasta/kernel/verifier.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/verifier.dart b/pkg/front_end/lib/src/fasta/kernel/verifier.dart
index 91b691a8925c7732cdb76844cdb216afcbfa0de5..691b46351967123ad5bc2698c397b64464b8f539 100644
--- a/pkg/front_end/lib/src/fasta/kernel/verifier.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/verifier.dart
@@ -47,7 +47,9 @@ class FastaVerifyingVisitor extends VerifyingVisitor
   problem(TreeNode node, String details, {TreeNode context}) {
     context ??= this.context;
     VerificationError error = new VerificationError(context, node, details);
-    printUnexpected(Uri.parse(fileUri), node?.fileOffset ?? -1, "$error");
+    var uri = fileUri != null ? Uri.parse(fileUri) : null;

[ahe, 2017/05/22 11:49:04]

Why is fileUri null?

[Siggi Cherem (dart-lang), 2017/05/22 23:15:48]

I ran into this when hitting verification errors for dangling references
initially. Now I cannot easily reproduce this by tweaking the code to produce
those errors, so I don't have a repro to show. If you prefer, I can revert it so
we determine the cause whenever we run into it again.

[ahe, 2017/05/24 15:45:47]

My preference would be to revert this so we can fix the underlying problem if
this happens again.

+    var offset = (uri != null && node != null) ? node.fileOffset : -1;
+    printUnexpected(uri, offset, "$error");
     errors.add(error);
   }