Add Quality Selection to Playback Overlay

app/src/main/java/org/jellyfin/androidtv/constant/QualityProfiles.kt

@@ -0,0 +1,31 @@
+package org.jellyfin.androidtv.constant
+
+enum class QualityProfiles(val quality: String) {
+	Quality_0("0"),
+	Quality_120("120"),
+	Quality_110("110"),
+	Quality_100("100"),
+	Quality_90("90"),
+	Quality_80("80"),
+	Quality_70("70"),
+	Quality_60("60"),
+	Quality_50("50"),
+	Quality_40("40"),
+	Quality_30("30"),
+	Quality_20("20"),
+	Quality_15("15"),
+	Quality_10("10"),
+	Quality_5("5"),
+	Quality_3("3"),
+	Quality_2("2"),
+	Quality_1("1"),
+	Quality_072("0.72"),
+	Quality_042("0.42");
+
+
+	companion object {
+		private val mapping = values().associateBy(QualityProfiles::quality)
+		fun fromPreference(quality: String) = mapping[quality]
+	}
+
+}

app/src/main/java/org/jellyfin/androidtv/ui/playback/PlaybackController.java

@@ -1120,6 +1120,14 @@ public void stop() {
         }
     }
 
+    public void refreshStream() {
+        // get current timestamp first
+        refreshCurrentPosition();
+
+        stop();
+        play(mCurrentPosition);
+    }
+
     public void endPlayback(Boolean closeActivity) {
         if (closeActivity) mFragment.getActivity().finish();
         stop();

app/src/main/java/org/jellyfin/androidtv/ui/playback/VideoQualityController.kt

@@ -0,0 +1,32 @@
+package org.jellyfin.androidtv.ui.playback
+
+import org.jellyfin.androidtv.preference.UserPreferences
+import org.jellyfin.androidtv.preference.UserPreferences.Companion.maxBitrate
+import org.koin.java.KoinJavaComponent.get
+import org.jellyfin.androidtv.constant.QualityProfiles
+
+class VideoQualityController(
+	@Suppress("unused")
+	private val parentController: PlaybackController
+) {
+	companion object {
+		private var previousQualitySelection = QualityProfiles.fromPreference(
+				get<UserPreferences>(UserPreferences::class.java)
+						.get(maxBitrate))
+	}
+
+	var currentQuality = previousQualitySelection
+		@Suppress("unused")
+		set(value) {
+			val checkedVal = QualityProfiles.fromPreference(
+				get<UserPreferences>(UserPreferences::class.java)
+					.get(maxBitrate))
+
+			previousQualitySelection = checkedVal
+			field = checkedVal
+		}
+
+	init {
+		currentQuality = previousQualitySelection
+	}
+}

app/src/main/java/org/jellyfin/androidtv/ui/playback/overlay/CustomPlaybackTransportControlGlue.java

@@ -36,6 +36,7 @@
 import org.jellyfin.androidtv.ui.playback.overlay.action.PreviousLiveTvChannelAction;
 import org.jellyfin.androidtv.ui.playback.overlay.action.RecordAction;
 import org.jellyfin.androidtv.ui.playback.overlay.action.SelectAudioAction;
+import org.jellyfin.androidtv.ui.playback.overlay.action.SelectQualityAction;
 import org.jellyfin.androidtv.ui.playback.overlay.action.SettingAction;
 import org.jellyfin.androidtv.ui.playback.overlay.action.ZoomAction;
 import org.koin.java.KoinJavaComponent;
@@ -51,6 +52,7 @@ public class CustomPlaybackTransportControlGlue extends PlaybackTransportControl
     private PlaybackControlsRow.SkipNextAction skipNextAction;
     private SelectAudioAction selectAudioAction;
     private ClosedCaptionsAction closedCaptionsAction;
+    private SelectQualityAction selectQualityAction;
     private SettingAction settingAction;
     private PlaybackSpeedAction playbackSpeedAction;
     private ZoomAction zoomAction;
@@ -181,6 +183,8 @@ private void initActions(Context context) {
         closedCaptionsAction.setLabels(new String[]{context.getString(R.string.lbl_subtitle_track)});
         settingAction = new SettingAction(context, this);
         settingAction.setLabels(new String[]{context.getString(R.string.lbl_adjust)});
+        selectQualityAction = new SelectQualityAction(context, this, playbackController);
+        selectQualityAction.setLabels(new String[]{context.getString(R.string.lbl_quality_profile)});
         playbackSpeedAction = new PlaybackSpeedAction(context, this, playbackController);
         playbackSpeedAction.setLabels(new String[]{context.getString(R.string.lbl_playback_speed)});
         zoomAction = new ZoomAction(context, this);
@@ -258,6 +262,7 @@ void addMediaActions() {
 
         if (!isLiveTv()) {
             secondaryActionsAdapter.add(playbackSpeedAction);
+            secondaryActionsAdapter.add(selectQualityAction);
         }
 
 
@@ -307,6 +312,9 @@ public void onCustomActionClicked(Action action, View view) {
             // This is a hack, we should instead have onPlaybackParametersChanged call out to this
             // class to notify rather than poll. But communication is unidirectional at the moment:
             mHandler.postDelayed(mRefreshEndTime, 5000);  // 5 seconds
+        }  else if (action == selectQualityAction) {
+            getPlayerAdapter().getLeanbackOverlayFragment().setFading(false);
+            selectQualityAction.handleClickAction(playbackController, getPlayerAdapter().getLeanbackOverlayFragment(), getContext(), view);
         } else if (action == zoomAction) {
             getPlayerAdapter().getLeanbackOverlayFragment().setFading(false);
             zoomAction.handleClickAction(playbackController, getPlayerAdapter().getLeanbackOverlayFragment(), getContext(), view);

app/src/main/java/org/jellyfin/androidtv/ui/playback/overlay/action/SelectQualityAction.kt

@@ -0,0 +1,81 @@
+package org.jellyfin.androidtv.ui.playback.overlay.action
+
+import android.content.Context
+import android.view.Gravity
+import android.view.View
+import android.widget.PopupMenu
+import org.jellyfin.androidtv.R
+import org.jellyfin.androidtv.constant.QualityProfiles
+import org.jellyfin.androidtv.preference.UserPreferences
+import org.jellyfin.androidtv.ui.playback.PlaybackController
+import org.jellyfin.androidtv.ui.playback.VideoQualityController
+import org.jellyfin.androidtv.ui.playback.overlay.CustomPlaybackTransportControlGlue
+import org.jellyfin.androidtv.ui.playback.overlay.LeanbackOverlayFragment
+import org.koin.java.KoinJavaComponent
+
+class SelectQualityAction (
+	context: Context,
+	customPlaybackTransportControlGlue: CustomPlaybackTransportControlGlue,
+	playbackController: PlaybackController
+) : CustomAction(context, customPlaybackTransportControlGlue) {
+	private val qualityController = VideoQualityController(playbackController)
+	private val qualityProfiles = QualityProfiles.values()
+
+	init {
+		initializeWithIcon(R.drawable.ic_select_quality)
+	}
+
+	override fun handleClickAction(
+		playbackController: PlaybackController,
+		leanbackOverlayFragment: LeanbackOverlayFragment,
+		context: Context, view: View
+	) {
+		val qualityMenu = populateMenu(context, view, qualityController)
+
+		qualityMenu.setOnDismissListener { leanbackOverlayFragment.setFading(true) }
+
+		qualityMenu.setOnMenuItemClickListener { menuItem ->
+			KoinJavaComponent.get<UserPreferences>(UserPreferences::class.java).set(
+					UserPreferences.maxBitrate, qualityProfiles[menuItem.itemId].quality)
+			qualityController.currentQuality = QualityProfiles.fromPreference(
+				KoinJavaComponent.get<UserPreferences>(UserPreferences::class.java)
+					.get(UserPreferences.maxBitrate))
+			playbackController.refreshStream()
+			qualityMenu.dismiss()
+			true
+		}
+
+		qualityMenu.show()
+	}
+
+	private fun formatQuality(quality: String, context: Context): String {
+
+		val conv = quality.toDouble()
+
+		@Suppress("MagicNumber")
+		val value = when {
+			conv == 0.0 -> context.getString(R.string.bitrate_auto)
+			conv >= 1.0 -> context.getString(R.string.bitrate_mbit, conv)
+			else -> context.getString(R.string.bitrate_kbit, conv * 1000.0)
+		}
+
+		conv.toString().removeSuffix(".0") to value
+
+		return value
+	}
+
+	private fun populateMenu(
+		context: Context,
+		view: View,
+		qualityController: VideoQualityController
+	) = PopupMenu(context, view, Gravity.END).apply {
+		qualityProfiles.forEachIndexed { i, selected ->
+			// Since this is purely numeric data, coerce to en_us to keep the linter happy
+			menu.add(0, i, i, formatQuality(selected.quality, context))
+		}
+
+		menu.setGroupCheckable(0, true, true)
+		menu.getItem(qualityProfiles.indexOf(qualityController.currentQuality)).isChecked = true
+	}
+
+}

app/src/main/java/org/jellyfin/androidtv/ui/preference/screen/PlaybackPreferencesScreen.kt

@@ -2,6 +2,7 @@ package org.jellyfin.androidtv.ui.preference.screen
 
 import android.app.AlertDialog
 import org.jellyfin.androidtv.R
+import org.jellyfin.androidtv.constant.QualityProfiles
 import org.jellyfin.androidtv.preference.UserPreferences
 import org.jellyfin.androidtv.preference.constant.AudioBehavior
 import org.jellyfin.androidtv.preference.constant.NEXTUP_TIMER_DISABLED
@@ -96,20 +97,17 @@ class PlaybackPreferencesScreen : OptionsFragment() {
 			@Suppress("MagicNumber")
 			list {
 				setTitle(R.string.pref_max_bitrate_title)
-				entries = setOf(
-					0.0, // auto
-					120.0, 110.0, 100.0, // 100 >=
-					90.0, 80.0, 70.0, 60.0, 50.0, 40.0, 30.0, 21.0, 15.0, 10.0, // 10 >=
-					5.0, 3.0, 2.0, 1.0, // 1 >=
-					0.72, 0.42 // 0 >=
-				).associate {
+				entries = enumValues<QualityProfiles>().associate {
+
+					val transform = it.quality.toDouble()
+
 					val value = when {
-						it == 0.0 -> getString(R.string.bitrate_auto)
-						it >= 1.0 -> getString(R.string.bitrate_mbit, it)
-						else -> getString(R.string.bitrate_kbit, it * 100.0)
+						transform == 0.0 -> getString(R.string.bitrate_auto)
+						transform >= 1.0 -> getString(R.string.bitrate_mbit, transform)
+						else -> getString(R.string.bitrate_kbit, transform * 1000.0)
 					}
 
-					it.toString().removeSuffix(".0") to value
+					transform.toString().removeSuffix(".0") to value
 				}
 				bind(userPreferences, UserPreferences.maxBitrate)
 			}

app/src/main/res/drawable/ic_select_quality.xml

@@ -0,0 +1,14 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:viewportWidth="24"
+    android:viewportHeight="24">
+    <path
+        android:pathData="M2.99,7.21c0,0,0-2.25,2.25-2.25h13.52c0,0,2.25,0,2.25,2.25v6.76c0,0,0,2.25-2.25,2.25h-4.51c0,0.75,0.09,1.31,0.28,1.69
+	h0.84c0.31,0,0.56,0.25,0.56,0.56c0,0.31-0.25,0.56-0.56,0.56H8.62c-0.31,0-0.56-0.25-0.56-0.56c0-0.31,0.25-0.56,0.56-0.56h0.84
+	c0.19-0.38,0.28-0.94,0.28-1.69H5.24c0,0-2.25,0-2.25-2.25V7.21z M4.56,6.25c-0.12,0.09-0.22,0.2-0.29,0.34
+	C4.18,6.79,4.12,7,4.11,7.22v6.75c0,0.37,0.09,0.57,0.16,0.68c0.08,0.12,0.19,0.21,0.34,0.29c0.19,0.09,0.4,0.15,0.61,0.16l0.03,0
+	h13.51c0.37,0,0.57-0.09,0.68-0.16c0.12-0.09,0.22-0.2,0.29-0.34c0.09-0.19,0.15-0.4,0.16-0.61l0-0.03V7.21
+	c0-0.37-0.09-0.57-0.16-0.68c-0.09-0.12-0.2-0.22-0.34-0.29c-0.2-0.1-0.41-0.15-0.63-0.16H5.24C4.87,6.09,4.67,6.17,4.56,6.25z"
+        android:fillColor="#fff" />
+</vector>

app/src/main/res/values/strings.xml

@@ -190,6 +190,7 @@
     <string name="lbl_fit">Normal</string>
     <string name="lbl_audio_track">Select audio track</string>
     <string name="lbl_playback_speed">Playback Speed</string>
+    <string name="lbl_quality_profile">Quality Profile</string>
     <string name="lbl_subtitle_track">Select subtitle track</string>
     <string name="msg_external_path">This feature will only work if you have properly set up your library on the server with network paths or path substitution and the client you are using can directly access these locations over the network.</string>
     <string name="btn_got_it">Got it</string>