Remove deprecated_internalProblem.

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

 // Copyright (c) 2016, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library fasta.scope;
 
 import 'builder/builder.dart' show Builder, TypeVariableBuilder;
 
-import 'deprecated_problems.dart'
-    show deprecated_InputError, deprecated_internalProblem;
+import 'deprecated_problems.dart' show deprecated_InputError;
+
+import 'problems.dart' show internalProblem, unsupported;
+
+import 'fasta_codes.dart' show messageInternalProblemExtendingUnmodifiableScope;
 
 class MutableScope {
   /// Names declared in this scope.
   Map<String, Builder> local;
 
   /// Setters declared in this scope.
   Map<String, Builder> setters;
 
   /// The scope that this scope is nested within, or `null` if this is the top
   /// level scope.
@@ skipping 22 matching lines @@
             isModifiable: isModifiable);
 
   Scope.immutable()
       : this(const <String, Builder>{}, const <String, Builder>{}, null,
             isModifiable: false);
 
   Scope.nested(Scope parent, {bool isModifiable: true})
       : this(<String, Builder>{}, null, parent, isModifiable: isModifiable);
 
   /// Don't use this. Use [becomePartOf] instead.
-  void set local(_) => deprecated_internalProblem("Unsupported operation.");
+  void set local(_) => unsupported("local=", -1, null);
 
   /// Don't use this. Use [becomePartOf] instead.
-  void set setters(_) => deprecated_internalProblem("Unsupported operation.");
+  void set setters(_) => unsupported("setters=", -1, null);
 
   /// Don't use this. Use [becomePartOf] instead.
-  void set parent(_) => deprecated_internalProblem("Unsupported operation.");
+  void set parent(_) => unsupported("parent=", -1, null);
 
   /// This scope becomes equivalent to [scope]. This is used for parts to
   /// become part of their library's scope.
   void becomePartOf(Scope scope) {
     assert(parent.parent == null);
     assert(scope.parent.parent == null);
     super.local = scope.local;
     super.setters = scope.setters;
     super.parent = scope.parent;
   }
@@ skipping 43 matching lines @@
   }
 
   Builder lookup(String name, int charOffset, Uri fileUri,
       {bool isInstanceScope: true}) {
     recordUse(name, charOffset, fileUri);
     Builder builder =
         lookupIn(name, charOffset, fileUri, local, isInstanceScope);
     if (builder != null) return builder;
     builder = lookupIn(name, charOffset, fileUri, setters, isInstanceScope);
     if (builder != null && !builder.hasProblem) {
-      return new deprecated_AccessErrorBuilder(
-          name, builder, charOffset, fileUri);
+      return new AccessErrorBuilder(name, builder, charOffset, fileUri);
     }
     if (!isInstanceScope) {
       // For static lookup, do not seach the parent scope.
       return builder;
     }
     return builder ?? parent?.lookup(name, charOffset, fileUri);
   }
 
   Builder lookupSetter(String name, int charOffset, Uri fileUri,
       {bool isInstanceScope: true}) {
     recordUse(name, charOffset, fileUri);
     Builder builder =
         lookupIn(name, charOffset, fileUri, setters, isInstanceScope);
     if (builder != null) return builder;
     builder = lookupIn(name, charOffset, fileUri, local, isInstanceScope);
     if (builder != null && !builder.hasProblem) {
-      return new deprecated_AccessErrorBuilder(
-          name, builder, charOffset, fileUri);
+      return new AccessErrorBuilder(name, builder, charOffset, fileUri);
     }
     if (!isInstanceScope) {
       // For static lookup, do not seach the parent scope.
       return builder;
     }
     return builder ?? parent?.lookupSetter(name, charOffset, fileUri);
   }
 
   bool hasLocalLabel(String name) => labels != null && labels.containsKey(name);
 
   void declareLabel(String name, Builder target) {
     if (isModifiable) {
       labels ??= <String, Builder>{};
       labels[name] = target;
     } else {
-      deprecated_internalProblem("Can't extend an unmodifiable scope.");
+      internalProblem(
+          messageInternalProblemExtendingUnmodifiableScope, -1, null);
     }
   }
 
   void forwardDeclareLabel(String name, Builder target) {
     declareLabel(name, target);
     forwardDeclaredLabels ??= <String, Builder>{};
     forwardDeclaredLabels[name] = target;
   }
 
   void claimLabel(String name) {
@@ skipping 20 matching lines @@
   deprecated_InputError declare(
       String name, Builder builder, int charOffset, Uri fileUri) {
     if (isModifiable) {
       if (usedNames?.containsKey(name) ?? false) {
         return new deprecated_InputError(
             fileUri, usedNames[name], "Previous use of '$name'.");
       }
       recordUse(name, charOffset, fileUri);
       local[name] = builder;
     } else {
-      deprecated_internalProblem("Can't extend an unmodifiable scope.");
+      internalProblem(
+          messageInternalProblemExtendingUnmodifiableScope, -1, null);
     }
     return null;
   }
 
   void merge(Scope scope,
       buildAmbiguousBuilder(String name, Builder existing, Builder member)) {
     Map<String, Builder> map = local;
 
     void mergeMember(String name, Builder member) {
       Builder existing = map[name];
@@ skipping 67 matching lines @@
   bool get hasProblem => true;
 
   String get deprecated_message;
 
   @override
   String get fullNameForErrors => name;
 }
 
 /// Represents a [builder] that's being accessed incorrectly. For example, an
 /// attempt to write to a final field, or to read from a setter.
-class deprecated_AccessErrorBuilder extends ProblemBuilder {
-  deprecated_AccessErrorBuilder(
-      String name, Builder builder, int charOffset, Uri fileUri)
+class AccessErrorBuilder extends ProblemBuilder {
+  AccessErrorBuilder(String name, Builder builder, int charOffset, Uri fileUri)
       : super(name, builder, charOffset, fileUri);
 
   Builder get parent => builder;
 
   bool get isFinal => builder.isFinal;
 
   bool get isField => builder.isField;
 
   bool get isRegularMethod => builder.isRegularMethod;
 
@@ skipping 13 matching lines @@
 
   String get deprecated_message => "Access error: '$name'.";
 }
 
 class AmbiguousBuilder extends ProblemBuilder {
   AmbiguousBuilder(String name, Builder builder, int charOffset, Uri fileUri)
       : super(name, builder, charOffset, fileUri);
 
   String get deprecated_message => "Duplicated named: '$name'.";
 }