<AWT Dev> Review Request for 8039081: [TEST_BUG] Test java/awt/TrayIcon/PopupMenuLeakTest/PopupMenuLeakTest.java fails

Ambarish Rapte ambarish.rapte at oracle.com
Mon Nov 30 15:44:42 UTC 2015

Hi Semyon,

                I have updated the fix patch with below changes,


1.       Code formatting.

2.       Reduced the number of OOMs.

3.       Removed un-required delays.


                Please review the update patch, which causes OOM early than previous change.








From: Semyon Sadetsky 
Sent: Monday, November 30, 2015 1:54 PM
To: Ambarish Rapte; Prasanta Sadhukhan; awt-dev at openjdk.java.net
Subject: Re: Review Request for 8039081: [TEST_BUG] Test java/awt/TrayIcon/PopupMenuLeakTest/PopupMenuLeakTest.java fails


Hi Ambarish,

On my Windows 7 64bit the test fails because of timeout.


On 11/20/2015 1:30 PM, Ambarish Rapte wrote:

Hi All,


                Please review the fix for JDK9,

                Bug: https://bugs.openjdk.java.net/browse/JDK-8039081

                Webrev: http://cr.openjdk.java.net/~arapte/8039081/webrev.00/




è The test fails on windows with below exception,


Caused by: java.lang.IllegalArgumentException: illegal popup menu container class 
        at sun.awt.windows.WPopupMenuPeer.<init>(WPopupMenuPeer.java:65) 
        at sun.awt.windows.WToolkit.createPopupMenu(WToolkit.java:461) 
        at java.awt.PopupMenu.addNotify(PopupMenu.java:124) 
        at PopupMenuLeakTest.addNotifyPopup(PopupMenuLeakTest.java:63)      


è With below exception on mac, ubuntu,


java.lang.OutOfMemoryError: Java heap space

                at PopupMenuLeakTest.assertCollected(PopupMenuLeakTest.java:85)

                at PopupMenuLeakTest.main(PopupMenuLeakTest.java:57)

                at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

                at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)

                at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)

                at java.lang.reflect.Method.invoke(Method.java:520)

                at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:92)

                at java.lang.Thread.run(Thread.java:747)



                This is a test code issue.

                On Windows Popup menu should have a valid parent container.

                Is there is no parent container , then above exception is thrown.


                And also, after throwing OOM exception, program tried to allocate new memory in catch.

                Which results in failure again.




                Updated the test code to add popup menu as a child of Frame.

                Verifiew on : Ubuntu, Windows & Mac



Many Thanks,


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20151130/201d0dde/attachment-0001.html>

More information about the awt-dev mailing list