RFR: 8322750: Test "api/java_awt/interactive/SystemTrayTests.html" failed because A blue ball icon is added outside of the system tray [v2]

Alexander Zvegintsev azvegint at openjdk.org
Thu Feb 15 12:51:15 UTC 2024


> There is an issue displaying the xembed icons in the appIndicators area which are not displayed correctly with certain Gnome Shell versions.
> It was already fixed [externally](https://gitlab.gnome.org/3v1n0/gnome-shell/-/commit/20a81d786697f40880e81d867453b1bad9524ec1).
> 
> However this is still a blocker on systems that have not received this fix, so this fix disables a SystemTray's support for Gnome Shell < 45 versions.
> 
> Gnome Shell version detection has the following logic:
> * execute `/usr/bin/gnome-shell --version`
> * parse its output to extract the major version
> * disable the SystemTray support if the version < 45 or parsing failed for some reason
> 
> 
> The numbering convention changed with the introduction of Gnome Shell 40.
> The old numbering convention is also handled correctly(e.g. [3.38.1](https://gitlab.gnome.org/GNOME/gnome-shell/-/blob/d15f6c75b19be1e32ec24165f09b3e74afaf7395/NEWS#L1134))
> 
> This is a simplified fix to make it easier to backport it.
> The improved solution will be to [receive the ShellVersion property via the DBUS API](https://unix.stackexchange.com/questions/73212/how-to-get-the-gnome-version/73225#73225)
> 
> Testing looks good:
> Oracle Linux 7.9, Gnome Shell 3.28.3
> Ubuntu 22.04, Gnome Shell 42.9
> Ubuntu 23.04, Gnome Shell 44.3
> Ubuntu 23.10, Gnome Shell 45.0
> Fedora 38, Gnome Shell 44.0

Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:

  IllegalThreadStateException

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/17860/files
  - new: https://git.openjdk.org/jdk/pull/17860/files/7e1b315b..006d6efd

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17860&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17860&range=00-01

  Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/17860.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17860/head:pull/17860

PR: https://git.openjdk.org/jdk/pull/17860


More information about the client-libs-dev mailing list