Remove specific RFI opt-in for Sabrina since it's supported out of the box in Android 12

This commit is contained in:
Cameron Gutman
2022-11-13 19:10:35 -06:00
parent 26bff28e4d
commit 33d484b7d1

View File

@@ -77,12 +77,6 @@ public class MediaCodecHelper {
refFrameInvalidationHevcPrefixes.add("omx.exynos");
refFrameInvalidationHevcPrefixes.add("c2.exynos");
// The Chromecast with Google TV 4K works well with HEVC RFI since we also use the
// vendor.low-latency.enable option.
if (Build.DEVICE.equalsIgnoreCase("sabrina")) {
refFrameInvalidationHevcPrefixes.add("omx.amlogic");
}
// Qualcomm and NVIDIA may be added at runtime
}
@@ -183,6 +177,9 @@ public class MediaCodecHelper {
// vendor.low-latency.enable. We will still use HEVC if decoderCanMeetPerformancePointWithHevcAndNotAvc()
// determines it's the only way to meet the performance requirements.
//
// With the Android 12 update, Sabrina now uses HEVC (with RFI) based upon FEATURE_LowLatency
// support, which provides equivalent latency to H.264 now.
//
// FIXME: Should we do this for all Amlogic S905X SoCs?
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P && !Build.DEVICE.equalsIgnoreCase("sabrina")) {
whitelistedHevcDecoders.add("omx.amlogic");