Wait for HandlerThread to terminate after calling quit.

Calling HandlerThread.quit() or .quitSafely() doesn't immediately terminate
the thread. It just instructs the Looper not to accept any new messages and
to terminate at the next opportunity. Added a HandlerThread.join() everywhere
where the intention is to close and release all resources and to stop all
threads.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=171525241

library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java

@@ -304,7 +304,6 @@ import java.io.IOException;
       // Restore the interrupted status.
       // Restore the interrupted status.
       Thread.currentThread().interrupt();
       Thread.currentThread().interrupt();
     }
     }
-    internalPlaybackThread.quit();
   }
   }
 
 
   public Looper getPlaybackLooper() {
   public Looper getPlaybackLooper() {
@@ -840,6 +839,7 @@ import java.io.IOException;
     resetInternal(true);
     resetInternal(true);
     loadControl.onReleased();
     loadControl.onReleased();
     setState(Player.STATE_IDLE);
     setState(Player.STATE_IDLE);
+    internalPlaybackThread.quit();
     synchronized (this) {
     synchronized (this) {
       released = true;
       released = true;
       notifyAll();
       notifyAll();

testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeSimpleExoPlayer.java

@@ -212,9 +212,17 @@ public class FakeSimpleExoPlayer extends SimpleExoPlayer {
     }
     }
 
 
     @Override
     @Override
+    @SuppressWarnings("ThreadJoinLoop")
     public void release() {
     public void release() {
       stop();
       stop();
       playbackThread.quitSafely();
       playbackThread.quitSafely();
+      while (playbackThread.isAlive()) {
+        try {
+          playbackThread.join();
+        } catch (InterruptedException e) {
+          // Ignore interrupt.
+        }
+      }
     }
     }
 
 
     @Override
     @Override