fix: access context safely when updating playback speed (#767)

* fix: access context safely when updating playback speed

* fix: prevent java.lang.IllegalArgumentException when skipping

Author: naveensingh

CHANGELOG.md

@@ -6,7 +6,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 ### Changed
-- Restored ability to show/hide notch area ([#749])      
+- Restored ability to show/hide notch area ([#749])     
 
 ### Fixed
 - Fixed overlap between editor controls and preview ([#752])

app/src/main/kotlin/org/fossify/gallery/fragments/VideoFragment.kt

@@ -642,13 +642,12 @@ class VideoFragment : ViewPagerFragment(), TextureView.SurfaceTextureListener,
 
             val drawableId =
                 if (isSlow) R.drawable.ic_playback_speed_slow_vector else R.drawable.ic_playback_speed_vector
-            binding.bottomVideoTimeHolder.videoPlaybackSpeed
-                .setDrawablesRelativeWithIntrinsicBounds(
-                    AppCompatResources.getDrawable(
-                        requireContext(),
-                        drawableId
+            context?.let {
+                binding.bottomVideoTimeHolder.videoPlaybackSpeed
+                    .setDrawablesRelativeWithIntrinsicBounds(
+                        AppCompatResources.getDrawable(it, drawableId)
                     )
-                )
+            }
         }
 
         @SuppressLint("SetTextI18n")
@@ -677,7 +676,7 @@ class VideoFragment : ViewPagerFragment(), TextureView.SurfaceTextureListener,
         val curr = mExoPlayer!!.currentPosition
         var newPosition =
             if (forward) curr + FAST_FORWARD_VIDEO_MS else curr - FAST_FORWARD_VIDEO_MS
-        newPosition = newPosition.coerceIn(0, mExoPlayer!!.duration)
+        newPosition = newPosition.coerceIn(0, maxOf(mExoPlayer!!.duration, 0))
         setPosition(newPosition)
         if (!mIsPlaying) {
             togglePlayPause()