From 2e0359908ca26ca7d3db25fb970b2aec30b05354 Mon Sep 17 00:00:00 2001
From: Harry <109690906+harry-signal@users.noreply.github.com>
Date: Mon, 6 May 2024 10:06:44 -0700
Subject: [PATCH] Don't treat view once looping gifs as videos

---
 .../ViewOnceMessageViewController.swift       | 24 +++++++++----------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/Signal/src/ViewControllers/ViewOnceMessageViewController.swift b/Signal/src/ViewControllers/ViewOnceMessageViewController.swift
index d05d300b3d..149f330332 100644
--- a/Signal/src/ViewControllers/ViewOnceMessageViewController.swift
+++ b/Signal/src/ViewControllers/ViewOnceMessageViewController.swift
@@ -121,20 +121,18 @@ class ViewOnceMessageViewController: OWSViewController {
             }
 
             let viewOnceType: Content.ContentType
-            if attachmentRef.renderingFlag == .shouldLoop {
+            switch attachmentStream.computeContentType() {
+            case .animatedImage:
+                viewOnceType = .animatedImage
+            case .image:
+                viewOnceType = .stillImage
+            case .video where attachmentRef.renderingFlag == .shouldLoop:
                 viewOnceType = .loopingVideo
-            } else {
-                switch attachmentStream.computeContentType() {
-                case .animatedImage:
-                    viewOnceType = .animatedImage
-                case .image:
-                    viewOnceType = .stillImage
-                case .video:
-                    viewOnceType = .video
-                case .audio, .file:
-                    owsFailDebug("Unexpected content type.")
-                    return
-                }
+            case .video:
+                viewOnceType = .video
+            case .audio, .file:
+                owsFailDebug("Unexpected content type.")
+                return
             }
 
             // To ensure that we never show the content more than once,