Only make AndroidDataConverters statically

Apparently, making one for each target creates memory issues.

RELNOTES: none
PiperOrigin-RevId: 196275764

Author: asteinb

src/main/java/com/google/devtools/build/lib/rules/android/AndroidDataConverter.java

@@ -108,6 +108,12 @@ public String map(T t) {
         .collect(Collectors.joining(joinerType.itemSeparator));
   }
 
+  /**
+   * Creates a builder for a new {@link AndroidDataConverter}.
+   *
+   * <p>Because of how Bazel handles these objects, call this method *only* as part of creating a
+   * static final field.
+   */
   public static <T> Builder<T> builder(JoinerType joinerType) {
     return new Builder<>(joinerType);
   }

src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java

@@ -33,6 +33,15 @@
 /** Builds up the spawn action for $android_rclass_generator. */
 public class RClassGeneratorActionBuilder {
 
+  private static final AndroidDataConverter<ValidatedAndroidData> AAPT_CONVERTER =
+      AndroidDataConverter.<ValidatedAndroidData>builder(JoinerType.COLON_COMMA)
+          .with(chooseDepsToArg(AndroidAaptVersion.AAPT))
+          .build();
+  private static final AndroidDataConverter<ValidatedAndroidData> AAPT2_CONVERTER =
+      AndroidDataConverter.<ValidatedAndroidData>builder(JoinerType.COLON_COMMA)
+          .with(chooseDepsToArg(AndroidAaptVersion.AAPT2))
+          .build();
+
   private final RuleContext ruleContext;
   private ResourceDependencies dependencies;
 
@@ -98,10 +107,7 @@ private void build(Artifact rTxt, ProcessedAndroidManifest manifest) {
         builder.addAll(
             VectorArg.addBefore("--library")
                 .each(dependencies.getResourceContainers())
-                .mapped(
-                    AndroidDataConverter.<ValidatedAndroidData>builder(JoinerType.COLON_COMMA)
-                        .with(chooseDepsToArg(version))
-                        .build()));
+                .mapped(version == AndroidAaptVersion.AAPT2 ? AAPT2_CONVERTER : AAPT_CONVERTER));
         if (version == AndroidAaptVersion.AAPT2) {
           inputs.addTransitive(dependencies.getTransitiveAapt2RTxt());
         } else {