Re: [10] JDK-8146537: TrayIcon Action Listener doesnt work in WIndows 10

2017-12-04 Thread shashidhara veerabhadraiah
Thank you Sergey for your review. Thanks and regards, Shashi > On 04-Dec-2017, at 1:03 PM, Sergey Bylokhov > wrote: > > Looks fine. > > On 27/11/2017 21:16, Shashidhara Veerabhadraiah wrote: >> Hi Sergey, Here is the Webrev containing fixes for your comments. >> http://cr.openjdk.java.net/~sv

Re: [10] JDK-8146537: TrayIcon Action Listener doesnt work in WIndows 10

2017-12-04 Thread Semyon Sadetsky
Hi Shashi, Shouldn't the BUTTON1_DOWN_MASK be added to the modifiers of the action? --Semyon On 11/27/2017 09:16 PM, Shashidhara Veerabhadraiah wrote: Hi Sergey, Here is the Webrev containing fixes for your comments. http://cr.openjdk.java.net/~sveerabhadra/8146537/webrev.01/ Thanks and rega

Re: [10] JDK-8146537: TrayIcon Action Listener doesnt work in WIndows 10

2017-12-04 Thread shashidhara veerabhadraiah
Hi Semyon, Thank for your review. BUTTON1_DOWN_MASK upon the GetJavaModifiers() provide if the left mouse button virtual key is pressed or not. Currently we are unable to capture left button click state using the ::GetKeyState(). The event NIN_BALLONUSERCLICK is generated only upon the left mou

[8u-dev] Request for approval and review for 8179665: [Windows] java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location

2017-12-04 Thread Alexey Ivanov
Hello, Could you please approve the following backport of the fix to 8u-dev? The patch from JDK9 applies cleanly except for copyright year updates; all code hunks apply cleanly. Semyon, Could you please review the changes? Webrev: http://cr.openjdk.java.net/~aivanov/8179665/jdk8/webrev.0/ JBS

Re: [8u-dev] Request for approval and review for 8179665: [Windows] java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location

2017-12-04 Thread Rob McKenna
Approved -Rob On 04/12/17 19:17, Alexey Ivanov wrote: > Hello, > > Could you please approve the following backport of the fix to 8u-dev? > > The patch from JDK9 applies cleanly except for copyright year updates; > all code hunks apply cleanly. > > Semyon, > > Could you please review the c

Re: [8u-dev] Request for approval and review for 8179665: [Windows] java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location

2017-12-04 Thread Rob McKenna
..pending review! -Rob On 04/12/17 21:33, Rob McKenna wrote: > Approved > > -Rob > > On 04/12/17 19:17, Alexey Ivanov wrote: > > Hello, > > > > Could you please approve the following backport of the fix to 8u-dev? > > > > The patch from JDK9 applies cleanly except for copyright year u

[10] Review Request: 8193012 Small cleanup of AWTEvent class

2017-12-04 Thread Sergey Bylokhov
Hello. Please review small cleanup for jdk10. Bug: https://bugs.openjdk.java.net/browse/JDK-8193012 Webrev can be found at: http://cr.openjdk.java.net/~serb/8193012/webrev.00 The fix for JDK-8129854[1] missed some fields in AWTEvent class which became unused. [1] http://hg.openjdk.java.net/j