Fix/remote params exception on app id update (#913)

* WIP - Fix for the remoteParams null exception on app id update
* Customer seeing issue with remoteParams being null in stacktrace exception after changing organization and app id of mobile app

Added 2 null (1 for app id and 1 for remoteParams) checks where null remoteParams exception occurs
* Need to validate if this is an issue with remoteParams being null for changed app ids
* Another issue would be if this is simply missing a null check on one of the routes to the null exception
* Attempting to reproduce this first before making an crazy changes

* Reproduced this issue with the following scenario:
1. Fresh install of app using OneSignal SDK with a valid app id and subscribed for push
2. New player is created in OneSignal app dashboard
3. Create an entirely new app on OneSignal dashboard
4. Delete the old app
5. Install updated app using OneSignal SDK with a new valid app id and subscribed for push
6. Crash occurs in OneSignal SDK because of a few possible scenarios (invalid app id or player id)
  * Basically we need to decide the correct way to handle this issue. The SDK should not have any other crashes now with the null checks in place, but should we add a backend patch also for old SDK users so it takes place now

* Fixed null check to account for a previous null emailId also
* This now follows the exact logic of OSSubscriptionState.setUserId

Author: mikechoch

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

@@ -62,7 +62,12 @@ void clearEmailAndId() {
     }
 
     void setEmailUserId(@NonNull String id) {
-        boolean changed = !id.equals(emailUserId);
+        boolean changed = false;
+        if (id == null)
+            changed = emailUserId != null;
+        else if (!id.equals(emailUserId))
+            changed = true;
+
         emailUserId = id;
         if (changed)
             observable.notifyChange(this);

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

@@ -9,10 +9,10 @@ static void setup(String appId, String userId, String adId) {
       if (opened)
          return;
 
-      if (OneSignal.remoteParams.enterprise)
+      if (OneSignal.remoteParams == null || OneSignal.remoteParams.enterprise)
          return;
 
-      if (userId == null)
+      if (appId == null || userId == null)
          return;
 
       String params = "?app_id=" + appId + "&user_id=" + userId;

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

@@ -111,7 +111,7 @@ static private void processJson(String json, final @NonNull CallBack callBack) {
          receiveReceiptEnabled = responseJson.optBoolean("receive_receipts_enable", false);
          outcomesParams = new OutcomesParams();
 
-         //Process outcomes params
+         // Process outcomes params
          if (responseJson.has(OUTCOME_PARAM)) {
             JSONObject outcomes = responseJson.optJSONObject(OUTCOME_PARAM);