PlayerHolder: inline setListener

It’s only used in this one place.

Author: Profpatsch

app/src/main/java/org/schabi/newpipe/player/helper/PlayerHolder.java

@@ -7,7 +7,6 @@
 import android.os.IBinder;
 import android.util.Log;
 
-import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.core.content.ContextCompat;
 
@@ -111,18 +110,6 @@ public void unsetListeners() {
         holderListener = null;
     }
 
-    public void setListener(@NonNull final PlayerServiceEventListener newListener,
-                            @NonNull final PlayerHolderLifecycleEventListener newHolderListener) {
-        listener = newListener;
-        holderListener = newHolderListener;
-
-        // Force reload data from service
-        if (player != null) {
-            holderListener.onServiceConnected(playerService, false);
-            player.setFragmentListener(internalListener);
-        }
-    }
-
     /**
      * Helper to handle context in common place as using the same
      * context to bind/unbind a service is crucial.
@@ -147,7 +134,14 @@ public void startService(final boolean playAfterConnect,
                              final PlayerHolderLifecycleEventListener newHolderListener
     ) {
         final Context context = getCommonContext();
-        setListener(newListener, newHolderListener);
+        listener = newListener;
+        holderListener = newHolderListener;
+
+        // Force reload data from service
+        if (player != null) {
+            holderListener.onServiceConnected(playerService, false);
+            player.setFragmentListener(internalListener);
+        }
         if (bound) {
             return;
         }