From 3310246351e5ae3ae3d72aa6adca41c3e3cd4edf Mon Sep 17 00:00:00 2001
From: Greyson Parrelli <greyson@signal.org>
Date: Tue, 28 Sep 2021 17:01:27 -0400
Subject: [PATCH] Inline MP4 GIF flag.

This reverts commit 91645e6adc8d4424692e349e4d87dc4e0c0f9fc6.
---
 .../securesms/giph/mp4/GiphyMp4PlaybackPolicy.java     |  4 ----
 .../securesms/giph/mp4/GiphyMp4Repository.java         | 10 +++-------
 .../java/org/thoughtcrime/securesms/mms/GifSlide.java  |  2 +-
 .../org/thoughtcrime/securesms/util/FeatureFlags.java  |  7 -------
 4 files changed, 4 insertions(+), 19 deletions(-)

diff --git a/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4PlaybackPolicy.java b/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4PlaybackPolicy.java
index 3385aa9b1..a6d71f61a 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4PlaybackPolicy.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4PlaybackPolicy.java
@@ -13,10 +13,6 @@ public final class GiphyMp4PlaybackPolicy {
 
   private GiphyMp4PlaybackPolicy() { }
 
-  public static boolean sendAsMp4() {
-    return FeatureFlags.mp4GifSendSupport();
-  }
-
   public static boolean autoplay() {
     return !DeviceProperties.isLowMemoryDevice(ApplicationDependencies.getApplication());
   }
diff --git a/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4Repository.java b/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4Repository.java
index f842b0610..1de8db3d6 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4Repository.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/giph/mp4/GiphyMp4Repository.java
@@ -51,19 +51,15 @@ final class GiphyMp4Repository {
 
   @WorkerThread
   private @NonNull Uri saveToBlobInternal(@NonNull GiphyImage giphyImage, boolean isForMms) throws IOException {
-    boolean sendAsMp4 = GiphyMp4PlaybackPolicy.sendAsMp4();
     String  url;
     String  mime;
 
-    if (sendAsMp4) {
-      url  = giphyImage.getMp4Url();
-      mime = MediaUtil.VIDEO_MP4;
-    } else if (isForMms) {
+    if (isForMms) {
       url  = giphyImage.getGifMmsUrl();
       mime = MediaUtil.IMAGE_GIF;
     } else {
-      url  = giphyImage.getGifUrl();
-      mime = MediaUtil.IMAGE_GIF;
+      url  = giphyImage.getMp4Url();
+      mime = MediaUtil.VIDEO_MP4;
     }
 
     Request request = new Request.Builder().url(url).build();
diff --git a/app/src/main/java/org/thoughtcrime/securesms/mms/GifSlide.java b/app/src/main/java/org/thoughtcrime/securesms/mms/GifSlide.java
index aa6beb902..9b3faede6 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/mms/GifSlide.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/mms/GifSlide.java
@@ -37,7 +37,7 @@ public class GifSlide extends ImageSlide {
                                               null,
                                               false,
                                               borderless,
-                                              FeatureFlags.mp4GifSendSupport(),
+                                              true,
                                               false));
 
     this.borderless = borderless;
diff --git a/app/src/main/java/org/thoughtcrime/securesms/util/FeatureFlags.java b/app/src/main/java/org/thoughtcrime/securesms/util/FeatureFlags.java
index 502c0ab48..439fad1b0 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/util/FeatureFlags.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/util/FeatureFlags.java
@@ -74,7 +74,6 @@ public final class FeatureFlags {
   private static final String ANIMATED_STICKER_MIN_TOTAL_MEMORY = "android.animatedStickerMinTotalMemory";
   private static final String MESSAGE_PROCESSOR_ALARM_INTERVAL  = "android.messageProcessor.alarmIntervalMins";
   private static final String MESSAGE_PROCESSOR_DELAY           = "android.messageProcessor.foregroundDelayMs";
-  private static final String MP4_GIF_SEND_SUPPORT              = "android.mp4GifSendSupport.2";
   private static final String MEDIA_QUALITY_LEVELS              = "android.mediaQuality.levels";
   private static final String RETRY_RECEIPT_LIFESPAN            = "android.retryReceiptLifespan";
   private static final String RETRY_RESPOND_MAX_AGE             = "android.retryRespondMaxAge";
@@ -114,7 +113,6 @@ public final class FeatureFlags {
       ANIMATED_STICKER_MIN_TOTAL_MEMORY,
       MESSAGE_PROCESSOR_ALARM_INTERVAL,
       MESSAGE_PROCESSOR_DELAY,
-      MP4_GIF_SEND_SUPPORT,
       MEDIA_QUALITY_LEVELS,
       RETRY_RECEIPT_LIFESPAN,
       RETRY_RESPOND_MAX_AGE,
@@ -168,7 +166,6 @@ public final class FeatureFlags {
       ANIMATED_STICKER_MIN_TOTAL_MEMORY,
       MESSAGE_PROCESSOR_ALARM_INTERVAL,
       MESSAGE_PROCESSOR_DELAY,
-      MP4_GIF_SEND_SUPPORT,
       MEDIA_QUALITY_LEVELS,
       RETRY_RECEIPT_LIFESPAN,
       RETRY_RESPOND_MAX_AGE,
@@ -356,10 +353,6 @@ public final class FeatureFlags {
     return getInteger(ANIMATED_STICKER_MIN_TOTAL_MEMORY, (int) ByteUnit.GIGABYTES.toMegabytes(3));
   }
 
-  public static boolean mp4GifSendSupport() {
-    return getBoolean(MP4_GIF_SEND_SUPPORT, false);
-  }
-
   public static @NonNull String getMediaQualityLevels() {
     return getString(MEDIA_QUALITY_LEVELS, "");
   }