From 5150ce10d5c9a8a72317fc42530b2a7df0a8f0a8 Mon Sep 17 00:00:00 2001
From: ajurkowski <ajurkowski@google.com>
Date: Tue, 25 Aug 2020 17:15:19 -0700
Subject: [PATCH] Remove dead code from `TreeArtifactValue`.

PiperOrigin-RevId: 328435075
---
 .../build/lib/skyframe/TreeArtifactValue.java | 31 -------------------
 1 file changed, 31 deletions(-)

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/TreeArtifactValue.java b/src/main/java/com/google/devtools/build/lib/skyframe/TreeArtifactValue.java
index df6359025bce1a..747df120dc8502 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/TreeArtifactValue.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/TreeArtifactValue.java
@@ -90,21 +90,6 @@ public interface MultiBuilder {
     MultiBuilder setArchivedRepresentation(
         ArchivedTreeArtifact archivedArtifact, FileArtifactValue metadata);
 
-    /**
-     * Make sure the builder will inject a {@link TreeArtifactValue} for a given {@linkplain
-     * SpecialArtifact tree artifact}.
-     *
-     * <p>Convenience method allowing to construct potentially empty {@link TreeArtifactValue} with
-     * a {@link MultiBuilder}.
-     *
-     * <p>There is no need to call this method before {@link #putChild(TreeFileArtifact,
-     * FileArtifactValue)} or {@link #setArchivedRepresentation(ArchivedTreeArtifact,
-     * FileArtifactValue)} since both methods implicitly make sure that the {@link
-     * TreeArtifactValue} will be created for the related {@linkplain SpecialArtifact parent tree
-     * artifact}.
-     */
-    MultiBuilder addTreeArtifact(SpecialArtifact treeArtifact);
-
     /**
      * For each unique parent seen by this builder, passes the aggregated metadata to {@link
      * TreeArtifactInjector#injectTree}.
@@ -494,14 +479,6 @@ public MultiBuilder setArchivedRepresentation(
       return this;
     }
 
-    @Override
-    public MultiBuilder addTreeArtifact(SpecialArtifact treeArtifact) {
-      Preconditions.checkArgument(
-          treeArtifact.isTreeArtifact(), "Not a tree artifact: %s", treeArtifact);
-      map.computeIfAbsent(treeArtifact, Builder::new);
-      return this;
-    }
-
     @Override
     public void injectTo(TreeArtifactInjector treeInjector) {
       map.forEach((parent, builder) -> treeInjector.injectTree(parent, builder.build()));
@@ -537,14 +514,6 @@ public MultiBuilder setArchivedRepresentation(
       return this;
     }
 
-    @Override
-    public MultiBuilder addTreeArtifact(SpecialArtifact treeArtifact) {
-      Preconditions.checkArgument(
-          treeArtifact.isTreeArtifact(), "Not a tree artifact: %s", treeArtifact);
-      children.computeIfAbsent(treeArtifact, ignored -> new ConcurrentHashMap<>());
-      return null;
-    }
-
     @Override
     public void injectTo(TreeArtifactInjector treeInjector) {
       children.forEach(