Re: RFR: JDK-8071469 Cleanup include and exclude of sound native libraries after source code restructure

2018-03-14 Thread Magnus Ihse Bursie
I forgot to add the client mailing lists as recipients. Sorry. (Not sure if "sounds" belong to "awt" or "2d".) /Magnus On 2018-03-15 02:07, Magnus Ihse Bursie wrote: From the bug description: Moving this to a separate bug from JDK-8055190. In SoundLibraries.gmk, the source code splitting is

Re: [8u-backport] Review request for 8195738 : scroll poistion in ScrollPane is reset after calling validate()

2018-03-14 Thread Patrick Chen
Hi, Why in src/windows/native/sun/windows/awt_ScrollPane.cpp did you delete the main condition? Means : the two variables are used isn't it ? Cheers, Pc 2018-03-14 11:02 GMT+01:00 Dipak Kumar : > Hi All, > > > > Please review the below patch (for 8u-backport) - > > > > Webrev : http://cr.openj

Re: [11][JDK-8152974] AWT hang occurs when sequenced events

2018-03-14 Thread Sergey Bylokhov
On 14/03/2018 07:53, Krishna Addepalli wrote: As per our discussion, I have modified the fix to process only SequencedEvents and ignore others. I have also modified the test case to include some dummy events between two SequencedEvents, and it works. Looks fine. However, could you point me

Re: [11][JDK-8152974] AWT hang occurs when sequenced events

2018-03-14 Thread Krishna Addepalli
Hi Sergey, As per our discussion, I have modified the fix to process only SequencedEvents and ignore others. I have also modified the test case to include some dummy events between two SequencedEvents, and it works. Here is the new webrev: http://cr.openjdk.java.net/~kaddepalli/8152974/webrev01/

[8u-backport] Review request for 8195738 : scroll poistion in ScrollPane is reset after calling validate()

2018-03-14 Thread Dipak Kumar
Hi All, Please review the below patch (for 8u-backport) - Webrev : http://cr.openjdk.java.net/~dkumar/8195738/webrev.00/ JBS : https://bugs.openjdk.java.net/browse/JDK-8195738 JDK 11 changeset - http://hg.openjdk.java.net/jdk/client/rev/96bebffe0be1 Patch pushed for JDK 11 appli