From a2a03aa0261561c20fa99a8241adbd61ae0655e5 Mon Sep 17 00:00:00 2001 From: Jae Seo Date: Wed, 11 May 2016 14:07:58 -0700 Subject: [PATCH] TIF: Remove the uniqueness check for track ID from notifyTracksChanged Requiring uniqueness of ID across all track types seems a little too much demand for developers and they do want to use the same ID on different track types. Also, this doesn't seem terribly useful. Hence remove the check. Bug: 28436618 Change-Id: I9040310c5ce31bcff522021d83f7508506cbd9d3 --- media/java/android/media/tv/TvInputService.java | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/media/java/android/media/tv/TvInputService.java b/media/java/android/media/tv/TvInputService.java index 210e62b37e728..ce4a453886296 100644 --- a/media/java/android/media/tv/TvInputService.java +++ b/media/java/android/media/tv/TvInputService.java @@ -401,20 +401,8 @@ public abstract class TvInputService extends Service { * with a different track ID. * * @param tracks A list which includes track information. - * @throws IllegalArgumentException if {@code tracks} contains redundant tracks. */ public void notifyTracksChanged(final List tracks) { - Set trackIdSet = new HashSet<>(); - for (TvTrackInfo track : tracks) { - String trackId = track.getId(); - if (trackIdSet.contains(trackId)) { - throw new IllegalArgumentException("redundant track ID: " + trackId); - } - trackIdSet.add(trackId); - } - trackIdSet.clear(); - - // TODO: Validate the track list. final List tracksCopy = new ArrayList<>(tracks); executeOrPostRunnableOnMainThread(new Runnable() { @MainThread