experimental UI: track touched packages during loading/analysis

To give a better understanding of which packages are on the critical
path during loading and analysis, provide information in the same way
as during execution: show the earliest started, but not yet completed
package. As not all packages looked at during the analysis phase are
reported to the progress receiver, use a custom class to aggregate those
data.

--
Change-Id: I03c25efdecb4124e1bc06fce8be9175dc56b5500
Reviewed-on: https://bazel-review.googlesource.com/#/c/3700
MOS_MIGRATED_REVID=123408689

1 files changed, 5 insertions, 13 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/LoadingPhaseStartedEvent.java b/src/main/java/com/google/devtools/build/lib/skyframe/LoadingPhaseStartedEvent.java
index e3e7444688..88bb44bfa9 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/LoadingPhaseStartedEvent.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/LoadingPhaseStartedEvent.java
@@ -13,31 +13,23 @@
 // limitations under the License.
 package com.google.devtools.build.lib.skyframe;
 
-import java.util.concurrent.atomic.AtomicInteger;
 /**
  * This event is fired at the beginning of the loading phase.
  */
 public final class LoadingPhaseStartedEvent {
 
-  final LoadingProgressReceiver loadingProgressReceiver;
-  final AtomicInteger numPackagesLoaded;
+  final PackageProgressReceiver packageProgress;
 
   /**
    * Construct the event
    *
    * @param loadingProgressReceiver a receiver that gets updated about the progress of loading
    */
-  public LoadingPhaseStartedEvent(
-      LoadingProgressReceiver loadingProgressReceiver, AtomicInteger numPackagesLoaded) {
-    this.loadingProgressReceiver = loadingProgressReceiver;
-    this.numPackagesLoaded = numPackagesLoaded;
+  public LoadingPhaseStartedEvent(PackageProgressReceiver packageProgress) {
+    this.packageProgress = packageProgress;
   }
 
-  public LoadingProgressReceiver getLoadingProgressReceiver() {
-    return loadingProgressReceiver;
-  }
-
-  public AtomicInteger getNumPackagesLoaded() {
-    return numPackagesLoaded;
+  public PackageProgressReceiver getPackageProgressReceiver() {
+    return packageProgress;
   }
 }