Fine. Fix for MultiplyDefinedElement in AST node.

Change-Id: I9d20ce86c1c1d3f7b28a9b33774fddf7a4a51a63
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/444502
Reviewed-by: Johnni Winther <[email protected]>
Commit-Queue: Konstantin Shcheglov <[email protected]>

Author: scheglov

pkg/analyzer/lib/src/fine/manifest_ast.dart

@@ -23,7 +23,8 @@ enum ManifestAstElementKind {
   formalParameter,
   importPrefix,
   typeParameter,
-  regular;
+  regular,
+  multiplyDefined;
 
   static final _bitCount = values.length.bitLength;
   static final _bitMask = (1 << _bitCount) - 1;
@@ -433,6 +434,9 @@ class _ElementCollector extends GeneralizingAstVisitor<void> {
       case DynamicElementImpl():
         kind = ManifestAstElementKind.dynamic_;
         rawIndex = 0;
+      case MultiplyDefinedElementImpl():
+        kind = ManifestAstElementKind.multiplyDefined;
+        rawIndex = 0;
       case FormalParameterElementImpl():
         kind = ManifestAstElementKind.formalParameter;
         rawIndex = indexOfFormalParameter(element);

pkg/analyzer/test/src/dart/analysis/driver_test.dart

@@ -44456,6 +44456,88 @@ const b = 0;
     );
   }
 
+  test_manifest_constInitializer_multiplyDefinedElement() async {
+    newFile('$testPackageLibPath/a.dart', r'''
+const foo = 0;
+''');
+
+    newFile('$testPackageLibPath/b.dart', r'''
+const foo = 0;
+''');
+
+    configuration.withElementManifests = true;
+    await _runLibraryManifestScenario(
+      initialCode: r'''
+import 'a.dart';
+import 'b.dart';
+const x = foo;
+''',
+      expectedInitialEvents: r'''
+[operation] linkLibraryCycle SDK
+[operation] linkLibraryCycle
+  package:test/a.dart
+    declaredGetters
+      foo: #M0
+        returnType: int @ dart:core
+    declaredVariables
+      foo: #M1
+        type: int @ dart:core
+        constInitializer
+          tokenBuffer: 0
+          tokenLengthList: [1]
+[operation] linkLibraryCycle
+  package:test/b.dart
+    declaredGetters
+      foo: #M2
+        returnType: int @ dart:core
+    declaredVariables
+      foo: #M3
+        type: int @ dart:core
+        constInitializer
+          tokenBuffer: 0
+          tokenLengthList: [1]
+[operation] linkLibraryCycle
+  package:test/test.dart
+    declaredGetters
+      x: #M4
+        returnType: InvalidType
+    declaredVariables
+      x: #M5
+        type: InvalidType
+        constInitializer
+          tokenBuffer: foo
+          tokenLengthList: [3]
+          elementIndexList
+            6 = multiplyDefined
+''',
+      updatedCode: r'''
+import 'a.dart';
+import 'b.dart';
+const x = foo;
+class A {}
+''',
+      expectedUpdatedEvents: r'''
+[operation] linkLibraryCycle
+  package:test/test.dart
+    declaredClasses
+      A: #M6
+        supertype: Object @ dart:core
+        interface: #M7
+    declaredGetters
+      x: #M4
+        returnType: InvalidType
+    declaredVariables
+      x: #M5
+        type: InvalidType
+        constInitializer
+          tokenBuffer: foo
+          tokenLengthList: [3]
+          elementIndexList
+            6 = multiplyDefined
+''',
+    );
+  }
+
   test_manifest_constInitializer_namedType() async {
     await _runLibraryManifestScenario(
       initialCode: r'''

pkg/analyzer/test/src/dart/analysis/result_printer.dart

@@ -1315,6 +1315,8 @@ class LibraryManifestPrinter {
                   sink.writelnWithIndent('$index = null');
                 case ManifestAstElementKind.dynamic_:
                   sink.writelnWithIndent('$index = dynamic');
+                case ManifestAstElementKind.multiplyDefined:
+                  sink.writelnWithIndent('$index = multiplyDefined');
                 case ManifestAstElementKind.formalParameter:
                   sink.writelnWithIndent('$index = formalParameter $rawIndex');
                 case ManifestAstElementKind.importPrefix: