From d4338bd64ac599dfec4aeec6d02c3ccccd3ecaec Mon Sep 17 00:00:00 2001
From: Niklas Merz <niklasmerz@apache.org>
Date: Thu, 2 Jan 2020 19:53:36 +0100
Subject: [PATCH] (android) improve defensive code for NULL check

---
 src/android/InAppBrowser.java | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/android/InAppBrowser.java b/src/android/InAppBrowser.java
index c35f3e8..8414438 100644
--- a/src/android/InAppBrowser.java
+++ b/src/android/InAppBrowser.java
@@ -313,7 +313,7 @@ public class InAppBrowser extends CordovaPlugin {
             this.cordova.getActivity().runOnUiThread(new Runnable() {
                 @Override
                 public void run() {
-                    if(dialog != null){
+                    if (dialog != null) {
                         dialog.show();
                     }
                 }
@@ -326,7 +326,7 @@ public class InAppBrowser extends CordovaPlugin {
             this.cordova.getActivity().runOnUiThread(new Runnable() {
                 @Override
                 public void run() {
-                    if(dialog != null){
+                    if (dialog != null) {
                         dialog.hide();
                     }
                 }
@@ -1069,17 +1069,15 @@ public class InAppBrowser extends CordovaPlugin {
                 lp.width = WindowManager.LayoutParams.MATCH_PARENT;
                 lp.height = WindowManager.LayoutParams.MATCH_PARENT;
 
-                dialog.setContentView(main);
-                if(dialog != null){
+                if (dialog != null) {
+                    dialog.setContentView(main);
                     dialog.show();
+                    dialog.getWindow().setAttributes(lp);
                 }
-                dialog.getWindow().setAttributes(lp);
                 // the goal of openhidden is to load the url and not display it
                 // Show() needs to be called to cause the URL to be loaded
-                if(openWindowHidden) {
-                    if(dialog != null){
-                        dialog.hide();
-                    }
+                if (openWindowHidden && dialog != null) {
+                    dialog.hide();
                 }
             }
         };