add Samsung S4 to hardware aec blacklist

Daniel Gultsch created 4 years ago

fixes #4267

Change summary

src/main/java/eu/siacs/conversations/xmpp/jingle/WebRTCWrapper.java | 6 
1 file changed, 4 insertions(+), 2 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/xmpp/jingle/WebRTCWrapper.java

@@ -64,8 +64,7 @@ public class WebRTCWrapper {
     private static final String EXTENDED_LOGGING_TAG = WebRTCWrapper.class.getSimpleName();
 
     private final ExecutorService executorService = Executors.newSingleThreadExecutor();
-
-    //we should probably keep this in sync with: https://github.com/signalapp/Signal-Android/blob/master/app/src/main/java/org/thoughtcrime/securesms/ApplicationContext.java#L296
+    
     private static final Set<String> HARDWARE_AEC_BLACKLIST = new ImmutableSet.Builder<String>()
             .add("Pixel")
             .add("Pixel XL")
@@ -79,6 +78,9 @@ public class WebRTCWrapper {
             .add("Redmi Note 5")
             .add("FP2") // Fairphone FP2
             .add("MI 5")
+            .add("GT-I9515") // Samsung Galaxy S4 Value Edition (jfvelte)
+            .add("GT-I9515L") // Samsung Galaxy S4 Value Edition (jfvelte)
+            .add("GT-I9505") // Samsung Galaxy S4 (jfltexx)
             .build();
 
     private static final int CAPTURING_RESOLUTION = 1920;