Change buildGestureListener visibility to protected for Java subclass access

Author: Ecomont

app/src/main/java/org/schabi/newpipe/player/ui/MainPlayerUi.java

@@ -146,8 +146,12 @@ public void setupAfterIntent() {
         }
     }
 
+    // `protected` matches the visibility declared in VideoPlayerUi.kt. The Kotlin parent used to
+    // declare this as `internal`, which the Kotlin compiler mangles to a JVM-private name that
+    // Java cannot see, so this class could not compile. Changing to `protected` gives the method
+    // a stable, unmangled JVM name accessible from Java subclasses.
     @Override
-    BasePlayerGestureListener buildGestureListener() {
+    protected BasePlayerGestureListener buildGestureListener() {
         return new MainPlayerGestureListener(this);
     }
 

app/src/main/java/org/schabi/newpipe/player/ui/PopupPlayerUi.java

@@ -101,8 +101,11 @@ public void setupAfterIntent() {
         initPopupCloseOverlay();
     }
 
+    // Same reason as MainPlayerUi: `protected` is required so this Java class can override the
+    // method declared in the Kotlin parent (VideoPlayerUi). Using `internal` in Kotlin produces
+    // a mangled JVM name that Java subclasses cannot reference.
     @Override
-    BasePlayerGestureListener buildGestureListener() {
+    protected BasePlayerGestureListener buildGestureListener() {
         return new PopupPlayerGestureListener(this);
     }