Fix not to call codec.setOnFrameRenderedListener(null, null) unless stoping codec

Refator to lift a method to create (and destroy) the OnFrameRenderedListenerV23() and split destroying the Handler from removing the MediaCodec onFrameRenderedListener to avoid double call to codec.setOnFrameRenderedListener()

library/core/src/main/java/com/google/android/exoplayer2/video/MediaCodecVideoRenderer.java

@@ -158,7 +158,7 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
 
 
   private boolean tunneling;
   private boolean tunneling;
   private int tunnelingAudioSessionId;
   private int tunnelingAudioSessionId;
-  /* package */ OnFrameRenderedListenerV23 tunnelingOnFrameRenderedListener;
+  /* package */ @Nullable OnFrameRenderedListenerV23 tunnelingOnFrameRenderedListener;
 
 
   private long lastInputTimeUs;
   private long lastInputTimeUs;
   private long outputStreamOffsetUs;
   private long outputStreamOffsetUs;
@@ -563,10 +563,7 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
     clearReportedVideoSize();
     clearReportedVideoSize();
     clearRenderedFirstFrame();
     clearRenderedFirstFrame();
     frameReleaseTimeHelper.disable();
     frameReleaseTimeHelper.disable();
-    if (tunnelingOnFrameRenderedListener != null) {
-      tunnelingOnFrameRenderedListener.destroyHandler(getCodec());
-    }
-    tunnelingOnFrameRenderedListener = null;
+    destroyOnFrameRenderListener();
     try {
     try {
       super.onDisabled();
       super.onDisabled();
     } finally {
     } finally {
@@ -690,10 +687,7 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
     }
     }
     codec.configure(mediaFormat, surface, crypto, 0);
     codec.configure(mediaFormat, surface, crypto, 0);
     if (Util.SDK_INT >= 23 && tunneling) {
     if (Util.SDK_INT >= 23 && tunneling) {
-      if (tunnelingOnFrameRenderedListener != null) {
-        tunnelingOnFrameRenderedListener.destroyHandler(codec);
-      }
-      tunnelingOnFrameRenderedListener = new OnFrameRenderedListenerV23(codec);
+      createOnFrameRenderedListener(codec);
     }
     }
   }
   }
 
 
@@ -1234,10 +1228,7 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
       MediaCodec codec = getCodec();
       MediaCodec codec = getCodec();
       // If codec is null then the listener will be instantiated in configureCodec.
       // If codec is null then the listener will be instantiated in configureCodec.
       if (codec != null) {
       if (codec != null) {
-        if (tunnelingOnFrameRenderedListener != null) {
-          tunnelingOnFrameRenderedListener.destroyHandler(codec);
-        }
-        tunnelingOnFrameRenderedListener = new OnFrameRenderedListenerV23(codec);
+        createOnFrameRenderedListener(codec);
       }
       }
     }
     }
   }
   }
@@ -1796,6 +1787,28 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
     return deviceNeedsSetOutputSurfaceWorkaround;
     return deviceNeedsSetOutputSurfaceWorkaround;
   }
   }
 
 
+  private void createOnFrameRenderedListener(MediaCodec codec) {
+    OnFrameRenderedListenerV23 oldListener = tunnelingOnFrameRenderedListener;
+    tunnelingOnFrameRenderedListener = new OnFrameRenderedListenerV23(codec);
+    if (oldListener != null) {
+      oldListener.destroyHandler();
+    }
+  }
+
+  private void destroyOnFrameRenderListener() {
+    if (tunnelingOnFrameRenderedListener != null) {
+      tunnelingOnFrameRenderedListener.destroyHandler();
+
+      MediaCodec codec = getCodec();
+      if (codec != null) {
+        tunnelingOnFrameRenderedListener.disable(codec);
+      }
+    }
+    tunnelingOnFrameRenderedListener = null;
+
+
+  }
+
   protected Surface getSurface() {
   protected Surface getSurface() {
     return surface;
     return surface;
   }
   }
@@ -1826,15 +1839,23 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
 
 
     /**
     /**
      * Cleanup any messages pending for this listener before removing references to it
      * Cleanup any messages pending for this listener before removing references to it
+     */
+    void destroyHandler() {
+      handler.removeCallbacksAndMessages(null);
+    }
+
+    /**
+     * Not required if we are switching one frame render listener for another and may cause
+     * overhead (MediaCodec calls native_enableOnFrameRenderedListener() twice then).  The
+     * stale event logic should prevent issues for a listener switch.
      *
      *
-     * @param codec optional, if the codec is still around it should be passed here
+     * This still might be a good idea when stopping the codec.
+     *
+     * @param codec codec that is being run down.
      */
      */
-    void destroyHandler(@Nullable MediaCodec codec) {
-      if (codec != null) {
+    private void disable(MediaCodec codec) {
       codec.setOnFrameRenderedListener(null, null);
       codec.setOnFrameRenderedListener(null, null);
     }
     }
-      handler.removeCallbacksAndMessages(null);
-    }
 
 
     @Override
     @Override
     public void onFrameRendered(MediaCodec codec, long presentationTimeUs, long nanoTime) {
     public void onFrameRendered(MediaCodec codec, long presentationTimeUs, long nanoTime) {