Fixed renderMode attr (#1263)
Somehow this was never implemented when adding the renderMode API
#1254
diff --git a/lottie/src/main/java/com/airbnb/lottie/LottieAnimationView.java b/lottie/src/main/java/com/airbnb/lottie/LottieAnimationView.java
index 41c4237..c79b29b 100644
--- a/lottie/src/main/java/com/airbnb/lottie/LottieAnimationView.java
+++ b/lottie/src/main/java/com/airbnb/lottie/LottieAnimationView.java
@@ -168,6 +168,14 @@
lottieDrawable.setScale(ta.getFloat(R.styleable.LottieAnimationView_lottie_scale, 1f));
}
+ if (ta.hasValue(R.styleable.LottieAnimationView_lottie_renderMode)) {
+ int renderModeOrdinal = ta.getInt(R.styleable.LottieAnimationView_lottie_renderMode, RenderMode.AUTOMATIC.ordinal());
+ if (renderModeOrdinal >= RenderMode.values().length) {
+ renderModeOrdinal = RenderMode.AUTOMATIC.ordinal();
+ }
+ renderMode = RenderMode.values()[renderModeOrdinal];
+ }
+
ta.recycle();
lottieDrawable.setSystemAnimationsAreEnabled(Utils.getAnimationScale(getContext()) != 0f);
@@ -861,12 +869,13 @@
}
private void enableOrDisableHardwareLayer() {
+ int layerType = LAYER_TYPE_SOFTWARE;
switch (renderMode) {
case HARDWARE:
- setLayerType(LAYER_TYPE_HARDWARE, null);
+ layerType = LAYER_TYPE_HARDWARE;
break;
case SOFTWARE:
- setLayerType(LAYER_TYPE_SOFTWARE, null);
+ layerType = LAYER_TYPE_SOFTWARE;
break;
case AUTOMATIC:
boolean useHardwareLayer = true;
@@ -874,11 +883,15 @@
useHardwareLayer = false;
} else if (composition != null && composition.getMaskAndMatteCount() > 4) {
useHardwareLayer = false;
+ } else if (Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) {
+ useHardwareLayer = false;
}
- setLayerType(useHardwareLayer ? LAYER_TYPE_HARDWARE : LAYER_TYPE_SOFTWARE, null);
+ layerType = useHardwareLayer ? LAYER_TYPE_HARDWARE : LAYER_TYPE_SOFTWARE;
break;
}
-
+ if (layerType != getLayerType()) {
+ setLayerType(layerType, null);
+ }
}
public boolean addLottieOnCompositionLoadedListener(@NonNull LottieOnCompositionLoadedListener lottieOnCompositionLoadedListener) {