Inject an event handler into the getErrorFreeVisitedPackages call.

This means we no longer need an event handler on the object itself.

--
MOS_MIGRATED_REVID=103982299

1 files changed, 3 insertions, 4 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeLabelVisitor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeLabelVisitor.java
index f325dc3ba3..1315fcf60d 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeLabelVisitor.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeLabelVisitor.java
@@ -56,7 +56,7 @@ final class SkyframeLabelVisitor implements TransitivePackageLoader {
   private Set<PackageIdentifier> errorFreeVisitedPackages;
   private Set<Label> visitedTargets;
   private Set<TransitiveTargetValue> previousBuildTargetValueSet = null;
-  private boolean lastBuildKeepGoing = false;
+  private boolean lastBuildKeepGoing;
   private final Multimap<Label, Label> rootCauses = HashMultimap.create();
 
   SkyframeLabelVisitor(SkyframeTransitivePackageLoader transitivePackageLoader,
@@ -229,15 +229,14 @@ final class SkyframeLabelVisitor implements TransitivePackageLoader {
     previousBuildTargetValueSet = currentBuildTargetValueSet;
   }
 
-
   @Override
   public Set<PackageIdentifier> getVisitedPackageNames() {
     return allVisitedPackages;
   }
 
   @Override
-  public Set<Package> getErrorFreeVisitedPackages() {
-    return transitivePackageLoader.retrievePackages(errorFreeVisitedPackages);
+  public Set<Package> getErrorFreeVisitedPackages(EventHandler eventHandler) {
+    return transitivePackageLoader.retrievePackages(eventHandler, errorFreeVisitedPackages);
   }
 
   /**