Removed debug printlns, tidied up some code

Author: beldenfox

modules/javafx.graphics/src/main/java/com/sun/javafx/scene/InputMethodStateManager.java

@@ -91,9 +91,6 @@ public InputMethodStateManager(Scene scene) {
      * Add a new Scene to the stack.
      */
     public void addScene(Scene scene) {
-        if (scenes.contains(scene)) {
-            System.err.println("Popup scene already present");
-        }
         scenes.addFirst(scene);
         updateInputMethodEventEnableState();
     }
@@ -102,10 +99,6 @@ public void addScene(Scene scene) {
      * Remove a Scene from the stack.
      */
     public void removeScene(Scene scene) {
-        if (scene != scenes.peekFirst()) {
-            System.err.println("Popup scene removed out of order");
-        }
-
         /**
          * If this scene is going away we should cleanup any composition
          * state. Hiding a window doesn't ensure proper cleanup.

modules/javafx.graphics/src/main/java/javafx/stage/PopupWindow.java

@@ -556,11 +556,11 @@ private void doVisibleChanging(boolean visible) {
      */
     private void doVisibleChanged(boolean visible) {
         final Window ownerWindowValue = getOwnerWindow();
+        Scene scene = getScene();
         if (visible) {
             rootWindow = getRootWindow(ownerWindowValue);
 
             startMonitorOwnerEvents(ownerWindowValue);
-            Scene scene = getScene();
             SceneHelper.getInputMethodStateManager(scene).addScene(scene);
             // currently we consider popup window to be focused when it is
             // visible and its owner window is focused (we need to track
@@ -574,7 +574,6 @@ private void doVisibleChanged(boolean visible) {
         } else {
             // This may generate events so it must be done while we're
             // still monitoring owner events.
-            Scene scene = getScene();
             SceneHelper.getInputMethodStateManager(scene).removeScene(scene);
             stopMonitorOwnerEvents(ownerWindowValue);
             unbindOwnerFocusedProperty(ownerWindowValue);