4.25 of 5: Writing of UnwrittenMergedAndroidData

Introduces the AndroidDataWriter and AndroidDataWritingVisitor to abstract the io operations from the data classes.
Necessary refactoring to the stubbed write method on DataValue in DataAsset.writeAsset and DataResource.writeResource.
New interface for the AttributeValues to reflect the simplifications of writing Resource Attributes.

Of special note is the fact all xml is written into a single file, values.xml. This is following the Gradle convention and aapt has demonstrated a preference of only reading a values.xml and ignoring all other xml files in the values directory. Unless profiling demonstrates an advantage to writing multiple files (which I doubt), this merger carries on this convention.

--
MOS_MIGRATED_REVID=119066611

1 files changed, 3 insertions, 4 deletions

diff --git a/src/tools/android/java/com/google/devtools/build/android/DataResourceXml.java b/src/tools/android/java/com/google/devtools/build/android/DataResourceXml.java
index ac5b392800..21e8c92657 100644
--- a/src/tools/android/java/com/google/devtools/build/android/DataResourceXml.java
+++ b/src/tools/android/java/com/google/devtools/build/android/DataResourceXml.java
@@ -58,7 +58,7 @@ public class DataResourceXml implements DataResource {
    * @throws FactoryConfigurationError Thrown with the {@link XMLInputFactory} is misconfigured.
    * @throws IOException Thrown when there is an error reading a file.
    */
-  public static void fromPath(
+  public static void parse(
       XMLInputFactory xmlInputFactory,
       Path path,
       Factory fqnFactory,
@@ -160,8 +160,7 @@ public class DataResourceXml implements DataResource {
   }
 
   @Override
-  public void write(Path newResourceDirectory) throws IOException {
-    // TODO(corysmith): Implement write.
-    throw new UnsupportedOperationException();
+  public void writeResource(FullyQualifiedName key, AndroidDataWritingVisitor mergedDataWriter) {
+    xml.write(key, source, mergedDataWriter);
   }
 }