Delete broken PriorityHandlerThread class

This class is unnecessary; you can simply pass a thread priority to HandlerThread's own constructor. It's also broken, since HandlerThread.run() immediately overrides the priority being set! ------------- Created by MOE: https:/google/moe MOE_MIGRATED_REVID=153461225
google · Apr 18, 2017 · ecb62cc · ecb62cc
1 parent 3c49044
commit ecb62cc
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 45 deletions.
diff --git a/library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java b/library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java
@@ -32,7 +32,6 @@
 import com.google.android.exoplayer2.trackselection.TrackSelectorResult;
 import com.google.android.exoplayer2.util.Assertions;
 import com.google.android.exoplayer2.util.MediaClock;
-import com.google.android.exoplayer2.util.PriorityHandlerThread;
 import com.google.android.exoplayer2.util.StandaloneMediaClock;
 import com.google.android.exoplayer2.util.TraceUtil;
 import java.io.IOException;
@@ -196,7 +195,7 @@ public ExoPlayerImplInternal(Renderer[] renderers, TrackSelector trackSelector,
 
  // Note: The documentation for Process.THREAD_PRIORITY_AUDIO that states "Applications can
  // not normally change to this priority" is incorrect.
- internalPlaybackThread = new PriorityHandlerThread("ExoPlayerImplInternal:Handler",
+ internalPlaybackThread = new HandlerThread("ExoPlayerImplInternal:Handler",
  Process.THREAD_PRIORITY_AUDIO);
  internalPlaybackThread.start();
  handler = new Handler(internalPlaybackThread.getLooper(), this);

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/util/PriorityHandlerThread.java b/library/core/src/main/java/com/google/android/exoplayer2/util/PriorityHandlerThread.java