[cfe] Split FragmentFactory from FragmentFactoryResult

This adds NativeMethodRegistry and CompilationUnitRegistry to decouple
the FragmentFactoryImpl from FragmentFactoryResult. FragmentFactoryResult
is removed and instead NativeMethodRegistry and CompilationUnitRegistry
is passed to the FragmentFactoryImpl to collect data during outline
building. This also avoids the need for storing the FragmentFactory
inside the SourceCompilationUnitImpl.

Change-Id: I58c475f22e6596d908cb4982716b7ed01dbbba62
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/438160
Reviewed-by: Chloe Stefantsova <[email protected]>
Commit-Queue: Johnni Winther <[email protected]>

Author: johnniwinther

pkg/front_end/lib/src/builder/compilation_unit.dart

@@ -19,7 +19,6 @@ import '../fragment/fragment.dart';
 import '../kernel/body_builder_context.dart';
 import '../source/name_space_builder.dart';
 import '../source/offset_map.dart';
-import '../source/source_class_builder.dart';
 import '../source/source_library_builder.dart';
 import '../source/source_loader.dart';
 import 'builder.dart';
@@ -149,7 +148,7 @@ abstract class SourceCompilationUnit
 
   String? get name;
 
-  int finishNativeMethods();
+  int finishNativeMethods(SourceLoader loader);
 
   String? get partOfName;
 
@@ -172,9 +171,6 @@ abstract class SourceCompilationUnit
 
   bool get mayImplementRestrictedTypes;
 
-  void takeMixinApplications(
-      Map<SourceClassBuilder, TypeBuilder> mixinApplications);
-
   void addDependencies(Library library, Set<SourceCompilationUnit> seen);
 
   /// Runs through all part directives in this compilation unit and adds the

pkg/front_end/lib/src/source/fragment_factory.dart

@@ -9,9 +9,7 @@ import 'package:kernel/ast.dart' hide Combinator, MapLiteralEntry;
 import '../api_prototype/lowering_predicates.dart';
 import '../base/combinator.dart' show CombinatorBuilder;
 import '../base/configuration.dart' show Configuration;
-import '../base/export.dart';
 import '../base/identifiers.dart' show Identifier;
-import '../base/import.dart';
 import '../base/modifiers.dart';
 import '../base/scope.dart';
 import '../builder/constructor_reference_builder.dart';
@@ -24,39 +22,8 @@ import '../builder/omitted_type_builder.dart';
 import '../builder/type_builder.dart';
 import '../fragment/fragment.dart';
 import 'offset_map.dart';
-import 'source_class_builder.dart';
-import 'source_library_builder.dart';
 import 'source_type_parameter_builder.dart';
 import 'type_parameter_factory.dart';
-import 'type_scope.dart';
-
-abstract class FragmentFactoryResult {
-  String? get name;
-
-  bool get isPart;
-
-  String? get partOfName;
-
-  Uri? get partOfUri;
-
-  /// The part directives in this compilation unit.
-  List<Part> get parts;
-
-  List<Import> get imports;
-
-  List<Export> get exports;
-
-  List<MetadataBuilder>? get metadata;
-
-  TypeScope get typeScope;
-
-  void takeMixinApplications(
-      Map<SourceClassBuilder, TypeBuilder> mixinApplications);
-
-  int finishNativeMethods();
-
-  List<LibraryPart> get libraryParts;
-}
 
 abstract class FragmentFactory {
   void beginClassOrNamedMixinApplicationHeader();