refactor (feat): Implement animated GIF / WEBP images

Author: dragonx943

app/src/main/kotlin/org/dokiteam/doki/core/util/ext/Coil.kt

@@ -82,25 +82,3 @@ fun SourceFetchResult.copyWithNewSource(): SourceFetchResult = SourceFetchResult
 	mimeType = mimeType,
 	dataSource = dataSource,
 )
-
-fun String.isAnimatedWebP(): Boolean = runCatching {
-    val client = okhttp3.OkHttpClient.Builder()
-        .connectTimeout(2, java.util.concurrent.TimeUnit.SECONDS)
-        .readTimeout(2, java.util.concurrent.TimeUnit.SECONDS)
-        .build()
-
-    val request = okhttp3.Request.Builder()
-        .url(this)
-        .header("Range", "bytes=0-20")
-        .build()
-
-    client.newCall(request).execute().use { response ->
-        val bytes = response.body.bytes()
-
-        bytes.size >= 21 &&
-            bytes[0] == 0x52.toByte() && bytes[3] == 0x46.toByte() &&
-            bytes[8] == 0x57.toByte() && bytes[11] == 0x50.toByte() &&
-            bytes[12] == 0x56.toByte() && bytes[15] == 0x58.toByte() &&
-            (bytes[20].toInt() and 0x02) != 0
-    }
-}.getOrDefault(false)

app/src/main/kotlin/org/dokiteam/doki/core/util/ext/String.kt

@@ -72,7 +72,7 @@ fun <T> Collection<T>.joinToStringWithLimit(context: Context, limit: Int, transf
 
 fun String.isHttpUrl() = startsWith("https://", ignoreCase = true) || startsWith("http://", ignoreCase = true)
 
-fun String.isAnimatedImage() = contains(".gif", ignoreCase = true) || contains(".webp", ignoreCase = true) && isAnimatedWebP()
+fun String.isSupportedAnimatedImage() = contains(".gif", ignoreCase = true) || contains(".webp", ignoreCase = true)
 
 fun concatStrings(context: Context, a: String?, b: String?): String? = when {
 	a.isNullOrEmpty() && b.isNullOrEmpty() -> null

app/src/main/kotlin/org/dokiteam/doki/image/ui/CoverImageView.kt

@@ -34,7 +34,7 @@ import org.dokiteam.doki.core.ui.image.TrimTransformation
 import org.dokiteam.doki.core.util.ext.bookmarkExtra
 import org.dokiteam.doki.core.util.ext.decodeRegion
 import org.dokiteam.doki.core.util.ext.getThemeColor
-import org.dokiteam.doki.core.util.ext.isAnimatedImage
+import org.dokiteam.doki.core.util.ext.isSupportedAnimatedImage
 import org.dokiteam.doki.core.util.ext.isNetworkError
 import org.dokiteam.doki.core.util.ext.mangaExtra
 import org.dokiteam.doki.core.util.ext.mangaSourceExtra
@@ -106,7 +106,7 @@ class CoverImageView @JvmOverloads constructor(
 		}
 	}
 
-	private fun isAnimatedUrl(url: String?): Boolean = url?.isAnimatedImage() == true
+	private fun isAnimatedUrl(url: String?): Boolean = url?.isSupportedAnimatedImage() == true
 
 	private fun newRequestBuilder(applyTrim: Boolean) = super.newRequestBuilder().apply {
 		if (trimImage && applyTrim) {

app/src/main/kotlin/org/dokiteam/doki/image/ui/ImageActivity.kt

@@ -38,7 +38,7 @@ import org.dokiteam.doki.core.util.ext.getDisplayIcon
 import org.dokiteam.doki.core.util.ext.getDisplayMessage
 import org.dokiteam.doki.core.util.ext.getParcelableExtraCompat
 import org.dokiteam.doki.core.util.ext.getThemeColor
-import org.dokiteam.doki.core.util.ext.isAnimatedImage
+import org.dokiteam.doki.core.util.ext.isSupportedAnimatedImage
 import org.dokiteam.doki.core.util.ext.mangaSourceExtra
 import org.dokiteam.doki.core.util.ext.observe
 import org.dokiteam.doki.core.util.ext.observeEvent
@@ -125,7 +125,7 @@ class ImageActivity : BaseActivity<ActivityImageBinding>(),
 
 	private fun loadImage() {
 		val url = intent.data?.toString()
-		val isAnimated = url?.isAnimatedImage() == true
+		val isAnimated = url?.isSupportedAnimatedImage() == true
 
 		if (isAnimated) {
 			viewBinding.ssiv.isVisible = false

app/src/main/kotlin/org/dokiteam/doki/reader/ui/pager/BasePageHolder.kt

@@ -23,7 +23,7 @@ import org.dokiteam.doki.core.image.CoilImageView
 import org.dokiteam.doki.core.os.NetworkState
 import org.dokiteam.doki.core.ui.list.lifecycle.LifecycleAwareViewHolder
 import org.dokiteam.doki.core.util.ext.getDisplayMessage
-import org.dokiteam.doki.core.util.ext.isAnimatedImage
+import org.dokiteam.doki.core.util.ext.isSupportedAnimatedImage
 import org.dokiteam.doki.core.util.ext.isLowRamDevice
 import org.dokiteam.doki.core.util.ext.isSerializable
 import org.dokiteam.doki.core.util.ext.observe
@@ -173,7 +173,7 @@ abstract class BasePageHolder<B : ViewBinding>(
 			bindingInfo.progressBar.isIndeterminate = true
 			bindingInfo.textViewStatus.setText(R.string.loading_)
 		}
-		val isAnimated = boundData?.url?.isAnimatedImage() == true
+		val isAnimated = boundData?.url?.isSupportedAnimatedImage() == true
 		when (state) {
 			is PageState.Converting -> {
 				bindingInfo.textViewStatus.setText(R.string.processing_)