Remove Runnable variables for Handlers.

[Isira-Seneviratne]

What is it?

• Bugfix (user facing)
• Feature (user facing)
• Codebase improvement (dev facing)
• Meta improvement to the project (dev facing)

Description of the changes in your PR

• Replace the Runnable variables being used in the downloader and double tap functionality with lambdas, and use String constants to keep track of them in the Handlers.

Fixes the following issue(s)

• Fixes #

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.

Due diligence

• I read the contribution guidelines.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

0.0% Coverage
0.0% Duplication

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

0.0% Coverage
0.0% Duplication

[Stypox]

You have added many unrelated changes (e.g. code style) and have not explained well what you did and how and why you did it. Please remove the unrelated changes and add a better PR description.

[Stypox]

Thank you, now I can see what's happening. I think it's better to use string constants instead of duplicating the strings (I'm referring to "doubleTap", "next", "show", which should be converted to DOUBLE_TAP, NEXT, SHOW).

[Stypox]

(the tests were failing)

/home/runner/work/NewPipe/NewPipe/app/src/main/java/org/schabi/newpipe/player/gesture/BasePlayerGestureListener.kt:40:1: Unexpected indentation (24) (should be 20) (indent)
/home/runner/work/NewPipe/NewPipe/app/src/main/java/org/schabi/newpipe/player/gesture/BasePlayerGestureListener.kt:73:1: Unexpected indentation (24) (should be 20) (indent)

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication