Don't strip AnimatedFactoryImpl and AnimatedFactoryImplSupport constr…

…uctors Summary: `AnimatedFactoryImpl` and `AnimatedFactoryImplSupport` currently have the `DoNotStrip` annotation. However, ProGuard strips their constructors since they are not used directly (we use reflection to instantiate them). By manually addin `DoNotStrip` to these constructors, we make sure that they are not stripped. Reviewed By: massimocarli, kirwan Differential Revision: D3458359 fbshipit-source-id: b5f020138ad82dc3225b99a55544edf939f206f2
facebook · Jun 21, 2016 · 0d0c575 · 0d0c575
1 parent 1ed36ff
commit 0d0c575
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/...src/main/java/com/facebook/imagepipeline/animated/factory/AnimatedFactoryImplSupport.java b/...src/main/java/com/facebook/imagepipeline/animated/factory/AnimatedFactoryImplSupport.java
@@ -26,6 +26,7 @@
 @DoNotStrip
 public class AnimatedFactoryImplSupport extends AnimatedFactoryImpl {
 
+  @DoNotStrip
   public AnimatedFactoryImplSupport(
       PlatformBitmapFactory platformBitmapFactory,
       ExecutorSupplier executorSupplier) {

diff --git a/...d-base/src/main/java/com/facebook/imagepipeline/animated/factory/AnimatedFactoryImpl.java b/...d-base/src/main/java/com/facebook/imagepipeline/animated/factory/AnimatedFactoryImpl.java
@@ -47,6 +47,7 @@ public class AnimatedFactoryImpl implements AnimatedFactory {
 
   private PlatformBitmapFactory mPlatformBitmapFactory;
 
+  @DoNotStrip
   public AnimatedFactoryImpl(
       PlatformBitmapFactory platformBitmapFactory,
       ExecutorSupplier executorSupplier) {