From 6fc024310602b7b4bd217e3ede649a4bef35e864 Mon Sep 17 00:00:00 2001
From: sheenachhabra <sheenachhabra@google.com>
Date: Wed, 26 Jun 2024 05:26:53 -0700
Subject: [PATCH] Use Set instead of a List for metadata collection

All metadata entries are supposed to be unique, so Set is more appropriate.

PiperOrigin-RevId: 646907916
---
 .../java/androidx/media3/muxer/MetadataCollector.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libraries/muxer/src/main/java/androidx/media3/muxer/MetadataCollector.java b/libraries/muxer/src/main/java/androidx/media3/muxer/MetadataCollector.java
index 0863405733..cb90e1394f 100644
--- a/libraries/muxer/src/main/java/androidx/media3/muxer/MetadataCollector.java
+++ b/libraries/muxer/src/main/java/androidx/media3/muxer/MetadataCollector.java
@@ -23,22 +23,22 @@ import androidx.media3.container.Mp4LocationData;
 import androidx.media3.container.Mp4OrientationData;
 import androidx.media3.container.Mp4TimestampData;
 import androidx.media3.container.XmpData;
-import java.util.ArrayList;
-import java.util.List;
+import java.util.HashSet;
+import java.util.Set;
 import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
 
 /** Collects and provides metadata: location, FPS, XMP data, etc. */
 /* package */ final class MetadataCollector {
   public Mp4OrientationData orientationData;
   public @MonotonicNonNull Mp4LocationData locationData;
-  public List<MdtaMetadataEntry> metadataEntries;
+  public Set<MdtaMetadataEntry> metadataEntries;
   public Mp4TimestampData timestampData;
   public @MonotonicNonNull XmpData xmpData;
 
   /** Creates an instance. */
   public MetadataCollector() {
     orientationData = new Mp4OrientationData(/* orientation= */ 0);
-    metadataEntries = new ArrayList<>();
+    metadataEntries = new HashSet<>();
     long currentTimeInMp4TimeSeconds = unixTimeToMp4TimeSeconds(System.currentTimeMillis());
     timestampData =
         new Mp4TimestampData(