Hi All,

 

Please review the patch for 8u-dev backport for HYPERLINK 
"https://bugs.openjdk.java.net/browse/JDK-8087516"JDK-8087516. It adds 
conditional support for gtk3. 

As part of the enhancement backport, we will be backporting 18 more bugs fixes 
done for GTK3 related issues since jdk9. Due to this, the backport will be 
pushed under https://bugs.openjdk.java.net/browse/JDK-8212147. More information 
about the 18 bugs can be found at 
https://bugs.openjdk.java.net/browse/JDK-8212147.

 

The https://bugs.openjdk.java.net/browse/JDK-8212147 is a re-creation of 
https://bugs.openjdk.java.net/browse/JDK-8206246. The fix for  
https://bugs.openjdk.java.net/browse/JDK-8206246 was backed out under 
https://bugs.openjdk.java.net/browse/JDK-8212139 due to some build issues. 
Those issues have been resolved and now this fix can be pushed.

 

Under https://bugs.openjdk.java.net/browse/JDK-8206246 the approved version was 
01 [1]. We have removed some un-used code, which was added by mistake as it was 
moved under gtk3 work and bug fixes. There is no change in functionality 
though. The updated webrev is version 02 [2]. Only following files have been 
changed in version 02 as compared to 01.

1.      modules/graphics/src/main/native-glass/gtk/glass_general.cpp

2.      modules/graphics/src/main/native-glass/gtk/glass_general.h

3.      modules/graphics/src/main/native-glass/gtk/wrapped.c

4.      modules/graphics/src/main/native-glass/gtk/wrapped.h

 

webrev: HYPERLINK 
"http://cr.openjdk.java.net/%7Epbansal/gtk3_JFX_backport/webrev.02/"http://cr.openjdk.java.net/~pbansal/gtk3_JFX_backport/webrev.02/

Enhancement under which this change will be pushed: 
https://bugs.openjdk.java.net/browse/JDK-8212147

JDK 9 Enhancement: https://bugs.openjdk.java.net/browse/JDK-8087516

JDK 9 Changeset: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/8c6d0386d3f5

 

[1] HYPERLINK 
"http://cr.openjdk.java.net/%7Epbansal/gtk3_JFX_backport/webrev.01/"http://cr.openjdk.java.net/~pbansal/gtk3_JFX_backport/webrev.01/

[2] HYPERLINK 
"http://cr.openjdk.java.net/%7Epbansal/gtk3_JFX_backport/webrev.02/"http://cr.openjdk.java.net/~pbansal/gtk3_JFX_backport/webrev.02/

 

 

Regards,

Pankaj

 

Reply via email to