Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-6624200

Regression test fails: test/closed/javax/swing/JMenuItem/4654927/bug4654927.java

    Details

    • Subcomponent:
    • Resolved In Build:
      b63
    • CPU:
      generic
    • OS:
      solaris

      Backports

        Description

        Regression test fails: test/closed/javax/swing/JMenuItem/4654927/bug4654927.java

        JDK: 7 b22
        Tested OSes:
        - Solaris nevada b70: test fails
        - Linux i686: test sometimes passes, sometimes fails
        - Windows XP: test passes

        Stack trace:
        java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location
        at java.awt.Component.getLocationOnScreen_NoTreeLock(Component.java:1838)
        at java.awt.Component.getLocationOnScreen(Component.java:1812)
        at JRobot.moveMouseTo(JRobot.java:90)
        at bug4654927.main(bug4654927.java:56)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:623)
        at com.sun.javatest.regtest.MainWrapper$MainThread.run(MainWrapper.java:78)
        at java.lang.Thread.run(Thread.java:675)

        The test logs in the attachment.

          Issue Links

            Activity

            Hide
            mlapshin Mikhail Lapshin (Inactive) added a comment -
            BT2:EVALUATION

            Looks like a threading issue.
            Show
            mlapshin Mikhail Lapshin (Inactive) added a comment - BT2:EVALUATION Looks like a threading issue.
            Hide
            dav Andrei Dmitriev (Inactive) added a comment -
            BT2:EVALUATION

            A thread condition is in the test. It posts some gui initialization on the EDT and simultaneously makes other testing stuff on the main thread which requires the gui being up and running. I'd rather align all these actions into a sequence.
            Show
            dav Andrei Dmitriev (Inactive) added a comment - BT2:EVALUATION A thread condition is in the test. It posts some gui initialization on the EDT and simultaneously makes other testing stuff on the main thread which requires the gui being up and running. I'd rather align all these actions into a sequence.
            Hide
            alexsch Alexander Scherbatiy (Inactive) added a comment -
            BT2:EVALUATION

            The main issue is in the jtreg which does not pass the GNOME_DESKTOP_SESSION_ID system variable to the tested JDK. The GNOME_DESKTOP_SESSION_ID system variable influence on the system L&F class definition.

            So if this variable is not set the MotifLookAndFeel is used as the system L&F on the Solaris system. The right behavior of the MotifLookAndFeel is closing a menu after clicking on the disabled menu item.

            So the suggested fix consists of:
            - Checking if system L&F class is Motif (this can happens if the GNOME_DESKTOP_SESSION_ID variable is not set on Solaris). In this case the behavior of clicking on the disabled menu item is different as fo others L&Fs.
            - Fixing jtreg that it pass the GNOME_DESKTOP_SESSION_ID system variable to the tested JDK
            Show
            alexsch Alexander Scherbatiy (Inactive) added a comment - BT2:EVALUATION The main issue is in the jtreg which does not pass the GNOME_DESKTOP_SESSION_ID system variable to the tested JDK. The GNOME_DESKTOP_SESSION_ID system variable influence on the system L&F class definition. So if this variable is not set the MotifLookAndFeel is used as the system L&F on the Solaris system. The right behavior of the MotifLookAndFeel is closing a menu after clicking on the disabled menu item. So the suggested fix consists of: - Checking if system L&F class is Motif (this can happens if the GNOME_DESKTOP_SESSION_ID variable is not set on Solaris). In this case the behavior of clicking on the disabled menu item is different as fo others L&Fs. - Fixing jtreg that it pass the GNOME_DESKTOP_SESSION_ID system variable to the tested JDK
            Hide
            alexsch Alexander Scherbatiy (Inactive) added a comment -
            BT2:EVALUATION

            There are also some issues with the disabled menu items in the Motif L&F
            - Dragging mouse to the disabled menu item does not close it
            - Clicking or dragging mouse to the disabled menu does not close it
            Show
            alexsch Alexander Scherbatiy (Inactive) added a comment - BT2:EVALUATION There are also some issues with the disabled menu items in the Motif L&F - Dragging mouse to the disabled menu item does not close it - Clicking or dragging mouse to the disabled menu does not close it
            Hide
            alexsch Alexander Scherbatiy (Inactive) added a comment -
            BT2:EVALUATION

            The jtrge issue has been resolved:
            7113599 jtreg should handle GNOME_DESKTOP_SESSION_ID
            Show
            alexsch Alexander Scherbatiy (Inactive) added a comment - BT2:EVALUATION The jtrge issue has been resolved: 7113599 jtreg should handle GNOME_DESKTOP_SESSION_ID
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/e0f91b40b8dd
            User: alexsch
            Date: 2012-10-23 10:32:35 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/e0f91b40b8dd User: alexsch Date: 2012-10-23 10:32:35 +0000
            Hide
            sgupta Shobhit Gupta added a comment -
            RULE closed/javax/swing/JMenuItem/4654927/bug4654927.java Exception java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location
            Show
            sgupta Shobhit Gupta added a comment - RULE closed/javax/swing/JMenuItem/4654927/bug4654927.java Exception java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location

              People

              • Assignee:
                alexsch Alexander Scherbatiy (Inactive)
                Reporter:
                mlapshin Mikhail Lapshin (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Imported:
                  Indexed: