Cleanup ValueVisitor (and dirty QuiescingExecutor)

Raises the level of abstraction of ValueVisitor's dependence on
AbstractQueueVisitor. Except for the "ForTestingOnly" methods now
available on the QuiescingExecutor interface, ValueVisitor is
agnostic to the implementation of its executor.

This also cleans up the full spectrum of visibility modifiers on
ValueVisitor methods, all of which ought to be private.

--
MOS_MIGRATED_REVID=106847453

1 files changed, 0 insertions, 9 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/concurrent/AbstractQueueVisitor.java b/src/main/java/com/google/devtools/build/lib/concurrent/AbstractQueueVisitor.java
index d5b6f1a7e9..b082bf20a9 100644
--- a/src/main/java/com/google/devtools/build/lib/concurrent/AbstractQueueVisitor.java
+++ b/src/main/java/com/google/devtools/build/lib/concurrent/AbstractQueueVisitor.java
@@ -574,15 +574,6 @@ public class AbstractQueueVisitor implements QuiescingExecutor {
     return (failFastOnInterrupt && isInterrupted()) || (unhandled != null && failFastOnException);
   }
 
-  /**
-   * Await interruption.  Used only in tests.
-   */
-  @VisibleForTesting
-  public boolean awaitInterruptionForTestingOnly(long timeout, TimeUnit units)
-      throws InterruptedException {
-    return interruptedLatch.await(timeout, units);
-  }
-
   /** Get latch that is released when exception is received by visitor. Used only in tests. */
   @VisibleForTesting
   public CountDownLatch getExceptionLatchForTestingOnly() {