Added proguard rules for observers

Author: jkasten2

OneSignalSDK/onesignal/consumer-proguard-rules.pro

@@ -8,6 +8,25 @@
 -keep public interface android.app.OnActivityPausedListener {*;}
 -keep class com.onesignal.ActivityLifecycleListenerCompat** {*;}
 
+-keep class com.onesignal.OSSubscriptionState {
+    void changed(com.onesignal.OSPermissionState);
+}
+
+-keep class com.onesignal.OSPermissionChangedInternalObserver {
+    void changed(com.onesignal.OSPermissionState);
+}
+
+-keep class com.onesignal.OSSubscriptionChangedInternalObserver {
+    void changed(com.onesignal.OSSubscriptionState);
+}
+
+-keep class ** implements com.onesignal.OSPermissionObserver {
+    void onOSPermissionChanged(com.onesignal.OSPermissionStateChanges);
+}
+
+-keep class ** implements com.onesignal.OSSubscriptionObserver {
+    void onOSSubscriptionChanged(com.onesignal.OSSubscriptionStateChanges);
+}
 
 -keep class com.onesignal.shortcutbadger.impl.AdwHomeBadger { <init>(...); }
 -keep class com.onesignal.shortcutbadger.impl.ApexHomeBadger { <init>(...); }

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSObservable.java

@@ -74,7 +74,7 @@ boolean notifyChange(final StateType state) {
          if (strongRefObserver != null) {
             try {
                Class<?> clazz = strongRefObserver.getClass();
-               final Method method = clazz.getMethod(methodName, state.getClass());
+               final Method method = clazz.getDeclaredMethod(methodName, state.getClass());
                method.setAccessible(true);
                if (fireOnMainThread) {
                   OSUtils.runOnMainUIThread(

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSPermissionChangedInternalObserver.java

@@ -28,9 +28,7 @@
 package com.onesignal;
 
 class OSPermissionChangedInternalObserver {
-   
-   // TODO: Make sure this method doesn't get prograuded out.
-   public void changed(OSPermissionState state) {
+   void changed(OSPermissionState state) {
       handleInternalChanges(state);
       fireChangesToPublicObserver(state);
    }

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSPermissionState.java

@@ -37,6 +37,8 @@ public class OSPermissionState implements Cloneable {
    OSObservable<Object, OSPermissionState> observable;
 
    OSPermissionState(boolean asFrom) {
+      // Java 8 method reference can be used in the future with Android Studio 2.4.0
+      //   OSPermissionChangedInternalObserver::changed
       observable = new OSObservable<>("changed", false);
 
       if (asFrom) {

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSSubscriptionChangedInternalObserver.java

@@ -28,7 +28,7 @@
 package com.onesignal;
 
 class OSSubscriptionChangedInternalObserver {
-   // TODO: Make sure this method doesn't get prograuded out.
+   
    public void changed(OSSubscriptionState state) {
       fireChangesToPublicObserver(state);
    }

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSSubscriptionState.java

@@ -60,7 +60,7 @@ public class OSSubscriptionState implements Cloneable {
    private String userId;
    private String pushToken;
 
-   public void changed(OSPermissionState state) {
+   void changed(OSPermissionState state) {
       setAccepted(state.getEnabled());
    }