Remove SkyframeExecutor#newPkgFunctionCache and `SkyframeExecutor#n…

…ewCompiledBuildFileCache`.

We no longer have a need to customize these caches.

PiperOrigin-RevId: 406143404

src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java

@@ -288,10 +288,10 @@ public abstract class SkyframeExecutor implements WalkableGraphFactory, Configur
   // package twice (first time loading to find load()ed bzl files and declare Skyframe
   // dependencies).
   private final Cache<PackageIdentifier, LoadedPackageCacheEntry> packageFunctionCache =
-      newPkgFunctionCache();
+      Caffeine.newBuilder().build();
   // Cache of parsed BUILD files, for PackageFunction. Same motivation as above.
   private final Cache<PackageIdentifier, PackageFunction.CompiledBuildFile> compiledBuildFileCache =
-      newCompiledBuildFileCache();
+      Caffeine.newBuilder().build();
 
   // Cache of parsed bzl files, for use when we're inlining BzlCompileFunction in
   // BzlLoadFunction. See the comments in BzlLoadFunction for motivations and details.
@@ -1197,15 +1197,6 @@ public void injectExtraPrecomputedValues(List<PrecomputedValue.Injected> extraPr
     }
   }
 
-  protected Cache<PackageIdentifier, LoadedPackageCacheEntry> newPkgFunctionCache() {
-    return Caffeine.newBuilder().build();
-  }
-
-  protected Cache<PackageIdentifier, PackageFunction.CompiledBuildFile>
-      newCompiledBuildFileCache() {
-    return Caffeine.newBuilder().build();
-  }
-
   private void setShowLoadingProgress(boolean showLoadingProgressValue) {
     showLoadingProgress.set(showLoadingProgressValue);
   }