From a47eb8a08a0ba4a75afe901be05bb8792e5279ea Mon Sep 17 00:00:00 2001
From: ibaker <ibaker@google.com>
Date: Mon, 12 Jul 2021 17:39:47 +0100
Subject: [PATCH] Explicitly override all non-deprecated methods in
 `Player.Listener`

Most of the super-interfaces are deprecated, but the intention is that
only the types are deprecated and the methods themselves shouldn't be.
In order to reflect this in javadoc we override all the methods in
`Player.Listener` in order to 'cancel' the deprecation.

This change deliberately doesn't override methods that are explicitly
deprecated with documented replacements (like
`Player.EventListener#onPlayerStateChanged`) - these should contine to
be marked as deprecated in javadoc.

PiperOrigin-RevId: 384253725
---
 .../com/google/android/exoplayer2/Player.java | 87 ++++++++++++++++++-
 1 file changed, 86 insertions(+), 1 deletion(-)

diff --git a/library/common/src/main/java/com/google/android/exoplayer2/Player.java b/library/common/src/main/java/com/google/android/exoplayer2/Player.java
index 558c930773..0bb973d4ef 100644
--- a/library/common/src/main/java/com/google/android/exoplayer2/Player.java
+++ b/library/common/src/main/java/com/google/android/exoplayer2/Player.java
@@ -325,6 +325,9 @@ public interface Player {
      * <p>State changes and events that happen within one {@link Looper} message queue iteration are
      * reported together and only after all individual callbacks were triggered.
      *
+     * <p>Only state changes represented by {@link EventFlags events} are reported through this
+     * method.
+     *
      * <p>Listeners should prefer this method over individual callbacks in the following cases:
      *
      * <ul>
@@ -700,12 +703,94 @@ public interface Player {
           DeviceListener,
           EventListener {
 
-    // For backward compatibility TextOutput and MetadataOutput must stay functional interfaces.
+    @Override
+    default void onTimelineChanged(Timeline timeline, @TimelineChangeReason int reason) {}
+
+    @Override
+    default void onMediaItemTransition(
+        @Nullable MediaItem mediaItem, @MediaItemTransitionReason int reason) {}
+
+    @Override
+    default void onTracksChanged(
+        TrackGroupArray trackGroups, TrackSelectionArray trackSelections) {}
+
+    @Override
+    default void onIsLoadingChanged(boolean isLoading) {}
+
+    @Override
+    default void onAvailableCommandsChanged(Commands availableCommands) {}
+
+    @Override
+    default void onPlaybackStateChanged(@State int state) {}
+
+    @Override
+    default void onPlayWhenReadyChanged(
+        boolean playWhenReady, @PlayWhenReadyChangeReason int reason) {}
+
+    @Override
+    default void onPlaybackSuppressionReasonChanged(
+        @PlaybackSuppressionReason int playbackSuppressionReason) {}
+
+    @Override
+    default void onIsPlayingChanged(boolean isPlaying) {}
+
+    @Override
+    default void onRepeatModeChanged(@RepeatMode int repeatMode) {}
+
+    @Override
+    default void onShuffleModeEnabledChanged(boolean shuffleModeEnabled) {}
+
+    @Override
+    default void onPlayerError(ExoPlaybackException error) {}
+
+    @Override
+    default void onPositionDiscontinuity(
+        PositionInfo oldPosition, PositionInfo newPosition, @DiscontinuityReason int reason) {}
+
+    @Override
+    default void onPlaybackParametersChanged(PlaybackParameters playbackParameters) {}
+
+    @Override
+    default void onAudioSessionIdChanged(int audioSessionId) {}
+
+    @Override
+    default void onAudioAttributesChanged(AudioAttributes audioAttributes) {}
+
+    @Override
+    default void onVolumeChanged(float volume) {}
+
+    @Override
+    default void onSkipSilenceEnabledChanged(boolean skipSilenceEnabled) {}
+
+    @Override
+    default void onDeviceInfoChanged(DeviceInfo deviceInfo) {}
+
+    @Override
+    default void onDeviceVolumeChanged(int volume, boolean muted) {}
+
+    @Override
+    default void onEvents(Player player, Events events) {}
+
+    @Override
+    default void onVideoSizeChanged(VideoSize videoSize) {}
+
+    @Override
+    default void onSurfaceSizeChanged(int width, int height) {}
+
+    @Override
+    default void onRenderedFirstFrame() {}
+
     @Override
     default void onCues(List<Cue> cues) {}
 
     @Override
     default void onMetadata(Metadata metadata) {}
+
+    @Override
+    default void onMediaMetadataChanged(MediaMetadata mediaMetadata) {}
+
+    @Override
+    default void onStaticMetadataChanged(List<Metadata> metadataList) {}
   }
 
   /**