Commit 9b4e9723 by Oliver Woodman

Don't use SEC VP8 decoder on Galaxy S3s.

The only Samsung devices with names starting "d2" that we're aware of
are Galaxy S3 variants, and also one Samsung Galaxy Pocket Neo d2aio
SAMSUNG-SGH-I747Z. This change speculatively includes that device too because
its name is very similar to SAMSUNG-SGH-I747 which is known to be affected.

Issue: #548
parent c4235d0e
...@@ -209,8 +209,10 @@ public final class MediaCodecUtil { ...@@ -209,8 +209,10 @@ public final class MediaCodecUtil {
return false; return false;
} }
// Work around an issue where the VP8 decoder on Samsung Galaxy S4 Mini does not render video. // Work around an issue where the VP8 decoder on Samsung Galaxy S3/S4 Mini does not render
if (Util.SDK_INT <= 19 && Util.DEVICE != null && Util.DEVICE.startsWith("serrano") // video.
if (Util.SDK_INT <= 19 && Util.DEVICE != null
&& (Util.DEVICE.startsWith("d2") || Util.DEVICE.startsWith("serrano"))
&& "samsung".equals(Util.MANUFACTURER) && name.equals("OMX.SEC.vp8.dec")) { && "samsung".equals(Util.MANUFACTURER) && name.equals("OMX.SEC.vp8.dec")) {
return false; return false;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment