Fine. Refactor LibraryManifestBuilder into multiple methods.

There are no functional changes (as evident from no changes to existing tests). But I noticed that we don't have tests for reusing IDs for instance getters inherited inside the same library cycles, so I added related tests.

Change-Id: I86ef590579e0cde879a32d1ab9643edb58f643eb
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/417321
Commit-Queue: Konstantin Shcheglov <[email protected]>
Reviewed-by: Paul Berry <[email protected]>

Author: scheglov

pkg/analyzer/lib/src/dart/analysis/library_context.dart

@@ -224,8 +224,7 @@ class LibraryContext {
               newLibraryManifests = LibraryManifestBuilder(
                 elementFactory: elementFactory,
                 inputLibraries: cycle.libraries,
-                inputLibraryManifests: inputLibraryManifests,
-                requirementsManifest: requirementsManifest,
+                inputManifests: inputLibraryManifests,
               ).computeManifests(
                 performance: performance,
               );