Merge pull request #61 from Iterable/bugfix/ITBL-6310-inapp-npe-fix

[ITBL-6310] Fix NPEs when in-apps are resized after being dismissed

Author: vbabenkoru

iterableapi/src/main/java/com/iterable/iterableapi/IterableInAppHTMLNotification.java

@@ -181,6 +181,11 @@ public void resize(final float height) {
         getOwnerActivity().runOnUiThread(new Runnable() {
             @Override
             public void run() {
+                // Since this is run asynchronously, notification might've been dismissed already
+                if (notification == null) {
+                    return;
+                }
+
                 DisplayMetrics displayMetrics = getOwnerActivity().getResources().getDisplayMetrics();
                 Window window = notification.getWindow();
                 Rect insetPadding = notification.insetPadding;

iterableapi/src/test/java/com/iterable/iterableapi/IterableInAppHTMLNotificationTest.java

@@ -0,0 +1,25 @@
+package com.iterable.iterableapi;
+
+import android.app.Activity;
+import android.graphics.Rect;
+
+import com.iterable.iterableapi.unit.BaseTest;
+
+import org.junit.Test;
+import org.robolectric.Robolectric;
+import org.robolectric.android.controller.ActivityController;
+
+public class IterableInAppHTMLNotificationTest extends BaseTest {
+
+    @Test
+    public void testDoNotCrashOnResizeAfterDismiss() {
+        ActivityController<Activity> controller = Robolectric.buildActivity(Activity.class).create().start();
+        Activity activity = controller.get();
+
+        IterableInAppManager.showIterableNotificationHTML(activity, "", "", null, 0.0, new Rect());
+
+        IterableInAppHTMLNotification notification = IterableInAppHTMLNotification.getInstance();
+        notification.dismiss();
+        notification.resize(500.0f);
+    }
+}