Document JavaSkylarkApiProvider static factory methods and remove one constructor.

PiperOrigin-RevId: 159557168

1 files changed, 7 insertions, 5 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java
index 7d799325da..de7dc82b41 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java
@@ -49,19 +49,21 @@ public final class JavaSkylarkApiProvider extends SkylarkApiProvider {
 
   @Nullable private final TransitiveInfoProviderMap transitiveInfoProviderMap;
 
+  /**
+   * Creates a Skylark API provider that reads information from its associated target's providers.
+   */
   public static JavaSkylarkApiProvider fromRuleContext() {
-    return new JavaSkylarkApiProvider();
+    return new JavaSkylarkApiProvider(null);
   }
 
+  /**
+   * Creates a Skylark API provider that reads information from an explicit provider map.
+   */
   public static JavaSkylarkApiProvider fromProviderMap(
       TransitiveInfoProviderMap transitiveInfoProviderMap) {
     return new JavaSkylarkApiProvider(transitiveInfoProviderMap);
   }
 
-  private JavaSkylarkApiProvider() {
-    this.transitiveInfoProviderMap = null;
-  }
-
   private JavaSkylarkApiProvider(TransitiveInfoProviderMap transitiveInfoProviderMap) {
     this.transitiveInfoProviderMap = transitiveInfoProviderMap;
   }