Use messages for (some) public API errors

pkg/front_end/lib/src/fasta/deprecated_problems.dart

Index: pkg/front_end/lib/src/fasta/deprecated_problems.dart
diff --git a/pkg/front_end/lib/src/fasta/deprecated_problems.dart b/pkg/front_end/lib/src/fasta/deprecated_problems.dart
index 084b837e921a95693d295b31a1479e3e389794e3..3c4223c2bb641f1473db13163eefdaf7a70f8f19 100644
--- a/pkg/front_end/lib/src/fasta/deprecated_problems.dart
+++ b/pkg/front_end/lib/src/fasta/deprecated_problems.dart
@@ -15,6 +15,8 @@ import 'colors.dart' show red;
 
 import 'messages.dart' show errorsAreFatal, deprecated_format, isVerbose;
 
+import 'fasta_codes.dart' show Message, LocatedMessage, Code;
+
 const String defaultServerAddress = "http://127.0.0.1:59410/";
 
 /// Tracks if there has been a crash reported through [reportCrash]. Should be
@@ -81,13 +83,23 @@ String colorError(String message) {
   return red(message);
 }
 
-class deprecated_InputError {
+class deprecated_InputError implements LocatedMessage {

[ahe, 2017/07/12 13:14:43]

I would like to maintain the invariant that instances of Message or
LocatedMessage are derived from a message in messages.yaml. So it would be great
if deprecated_InputError doesn't implement LocatedMessage.

This is how I handle the similar situation in verifier.dart:

  problem(TreeNode node, String details, {TreeNode context}) {
    context ??= this.context;
    VerificationError error = new VerificationError(context, node, details);
    CompilerContext.current.report(
        templateUnspecified
            .withArguments("$error")
            .withLocation(Uri.parse(fileUri), node?.fileOffset ?? -1),
        Severity.error);
    errors.add(error);
  }

CompilerContext.current.report is something that I've added based on what we
discussed in CL 2979713002, and should make it easier to hook up the reporting
you're adding with this CL.

[Siggi Cherem (dart-lang), 2017/07/12 21:54:59]

Done (removed the implements here)

   final Uri uri;
 
   final int charOffset;
 
   final Object error;
 
+  String get message => safeToString(error);
+
+  Message get messageObject => null;
+
+  Code get code => null;
+
+  Map<String, dynamic> get arguments => {};
+
+  String get tip => null;
+
   deprecated_InputError(this.uri, int charOffset, this.error)
       : this.charOffset = charOffset ?? -1;