Invoker null check

Author: prsadhuk

src/java.desktop/share/classes/javax/swing/JPopupMenu.java

@@ -952,20 +952,21 @@ && isVisible()) {
     @BeanProperty(bound = false, expert = true, description
             = "The invoking component for the popup menu")
     public void setInvoker(Component invoker) {
-        if (invoker != null) {
-            Component oldInvoker = this.invoker;
-            this.invoker = invoker;
+        Component oldInvoker = this.invoker;
+        this.invoker = invoker;
 
-            if ((oldInvoker != this.invoker) && (ui != null)) {
-                ui.uninstallUI(this);
-                if (oldInvoker != null) {
-                    oldInvoker.removePropertyChangeListener("ancestor", propListener);
-                }
+        if ((oldInvoker != this.invoker) && (ui != null)) {
+            ui.uninstallUI(this);
+            if (oldInvoker != null) {
+                oldInvoker.removePropertyChangeListener("ancestor", propListener);
+            }
+            if (invoker != null) {
                 invoker.addPropertyChangeListener("ancestor", propListener);
-                ui.installUI(this);
             }
-            invalidate();
+            ui.installUI(this);
         }
+        invalidate();
+
     }
 
     /**

test/jdk/javax/swing/JPopupMenu/TestPopupInvoker.java

@@ -73,6 +73,7 @@ public void popupMenuWillBecomeVisible(PopupMenuEvent e) {
             @Override
             public void popupMenuWillBecomeInvisible(PopupMenuEvent e) {
                 popupHidden.countDown();
+                popupMenu.setInvoker(null);
             }
 
             @Override
@@ -108,7 +109,6 @@ public static void main(String[] args) throws Exception {
             }
         } finally {
             SwingUtilities.invokeAndWait(() -> {
-                popupMenu.setInvoker(null);
                 if (frame != null) {
                     frame.dispose();
                 }