[8u-backport] JDK-8212147: [JavaFX] Conditional support for GTK 3 on Linux

[Pankaj Bansal]

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

[Kevin Rushforth]

+1

As with last time, this will need a second reviewer before pushing it to
8u-dev.

-- Kevin

Post by Pankaj Bansal
Hi All,
Please review the patch for 8u-dev backport for JDK-8087516
<https://bugs.openjdk.java.net/browse/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
http://cr.openjdk.java.net/~pbansal/gtk3_JFX_backport/webrev.02/
<http://cr.openjdk.java.net/%7Epbansal/gtk3_JFX_backport/webrev.02/>
https://bugs.openjdk.java.net/browse/JDK-8212147
JDK 9 Enhancement: https://bugs.openjdk.java.net/browse/JDK-8087516
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/8c6d0386d3f5
[1] http://cr.openjdk.java.net/~pbansal/gtk3_JFX_backport/webrev.01/
<http://cr.openjdk.java.net/%7Epbansal/gtk3_JFX_backport/webrev.01/>
[2] http://cr.openjdk.java.net/~pbansal/gtk3_JFX_backport/webrev.02/
<http://cr.openjdk.java.net/%7Epbansal/gtk3_JFX_backport/webrev.02/>
Regards,
Pankaj