Add static method for wrapping drawable factories

Reviewed By: oprisnik

Differential Revision: D64253193

vito/core-impl/src/main/java/com/facebook/fresco/vito/core/impl/KFrescoController.kt

@@ -69,6 +69,10 @@ class KFrescoController(
                   drawableFactory?.createDrawable(r, a, b)?.let { createDrawableModel(it, b) }
                 }
               }
+              ?: run {
+                FLog.e(TAG, "Could not create Drawable for CloseableImage: $b")
+                null
+              }
         }
       }
 

vito/drawee-support/src/main/java/com/facebook/fresco/vito/draweesupport/DrawableFactoryWrapper.kt

@@ -14,7 +14,7 @@ import com.facebook.fresco.vito.options.ImageOptionsDrawableFactory
 import com.facebook.imagepipeline.drawable.DrawableFactory
 import com.facebook.imagepipeline.image.CloseableImage
 
-class DrawableFactoryWrapper(private val drawableFactory: DrawableFactory) :
+class DrawableFactoryWrapper private constructor(private val drawableFactory: DrawableFactory) :
     ImageOptionsDrawableFactory {
   override fun createDrawable(
       resources: Resources,
@@ -30,4 +30,15 @@ class DrawableFactoryWrapper(private val drawableFactory: DrawableFactory) :
       } else {
         null
       }
+
+  companion object {
+    @JvmStatic
+    fun wrap(drawableFactory: DrawableFactory): ImageOptionsDrawableFactory {
+      return if (drawableFactory is ImageOptionsDrawableFactory) {
+        drawableFactory
+      } else {
+        DrawableFactoryWrapper(drawableFactory)
+      }
+    }
+  }
 }

vito/provider/src/main/java/com/facebook/fresco/vito/provider/impl/DefaultFrescoVitoProvider.kt

@@ -86,11 +86,8 @@ class DefaultFrescoVitoProvider(
       return if (animatedDrawableFactory == null) {
         bitmapFactory
       } else {
-        if (animatedDrawableFactory is ImageOptionsDrawableFactory) {
-          ArrayVitoDrawableFactory(bitmapFactory, animatedDrawableFactory)
-        } else {
-          ArrayVitoDrawableFactory(bitmapFactory, DrawableFactoryWrapper(animatedDrawableFactory))
-        }
+        ArrayVitoDrawableFactory(
+            bitmapFactory, DrawableFactoryWrapper.wrap(animatedDrawableFactory))
       }
     }
   }

vito/provider/src/main/java/com/facebook/fresco/vito/provider/impl/kotlin/KFrescoVitoProvider.kt

@@ -62,8 +62,8 @@ class KFrescoVitoProvider(
   override fun getConfig(): FrescoVitoConfig = vitoConfig
 
   private fun getFactory(): ImageOptionsDrawableFactory? {
-    return ImagePipelineFactory.getInstance().getAnimatedDrawableFactory(null)?.let {
-      DrawableFactoryWrapper(it)
-    }
+    return ImagePipelineFactory.getInstance()
+        .getAnimatedDrawableFactory(null)
+        ?.let(DrawableFactoryWrapper::wrap)
   }
 }