Skip to content

8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity) - #840

Open
satyenme wants to merge 2 commits into
openjdk:masterfrom
satyenme:8036915
Open

8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity)#840
satyenme wants to merge 2 commits into
openjdk:masterfrom
satyenme:8036915

Conversation

@satyenme

@satyenme satyenme commented Jul 10, 2026

Copy link
Copy Markdown

Backporting "JDK-8159460: On Ubuntu Unity, problem with java/awt/Window/FindOwner/FindOwnerTest" and "JDK-8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity)". Backporting to clear the way for https://bugs.openjdk.org/browse/JDK-8305825, which is pulled in by oracle, soft-requires this patch, and which solves a similar UI bug on the default Gnome UI.

JDK-8036915 and has a dependency on UNITY_COMPIZ_WM defined by JDK-8159460. Grouping these two changes together and omitting the change to jdk/test/java/awt/Window/FindOwner/FindOwnerTest.html (which isn't present in JDK 8).

For completeness tested the SetLocSwingDlgTest.java test on JDK-8036915 with Ubuntu Unity (testing with 24.10) to confirm passing with patch and failing without patch.



Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • JDK-8036915 needs maintainer approval
  • Commit message must refer to an issue
  • JDK-8159460 needs maintainer approval

Issues

  • JDK-8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity) (Bug - P3)
  • JDK-8159460: On Ubuntu Unity, problem with java/awt/Window/FindOwner/FindOwnerTest (Bug - P3)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk8u-dev.git pull/840/head:pull/840
$ git checkout pull/840

Update a local copy of the PR:
$ git checkout pull/840
$ git pull https://git.openjdk.org/jdk8u-dev.git pull/840/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 840

View PR using the GUI difftool:
$ git pr show -t 840

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk8u-dev/pull/840.diff

Using Webrev

Link to Webrev Comment

@bridgekeeper

bridgekeeper Bot commented Jul 10, 2026

Copy link
Copy Markdown

👋 Welcome back ssubramaniam! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk

openjdk Bot commented Jul 10, 2026

Copy link
Copy Markdown

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@openjdk openjdk Bot changed the title Backport 947d19a3a6359b3b95da39fccc68a657ab4c4db3 8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity) Jul 10, 2026
@openjdk

openjdk Bot commented Jul 10, 2026

Copy link
Copy Markdown

This backport pull request has now been updated with issue from the original commit.

@openjdk openjdk Bot added backport Port of a pull request already in a different code base rfr Pull request is ready for review labels Jul 10, 2026
@mlbridge

mlbridge Bot commented Jul 10, 2026

Copy link
Copy Markdown

Webrevs

@satyenme

Copy link
Copy Markdown
Author

/issue JDK-8159460

@openjdk

openjdk Bot commented Jul 10, 2026

Copy link
Copy Markdown

@satyenme
Adding additional issue to issue list: 8159460: On Ubuntu Unity, problem with java/awt/Window/FindOwner/FindOwnerTest.

@bridgekeeper

bridgekeeper Bot commented Aug 7, 2026

Copy link
Copy Markdown

@satyenme This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Port of a pull request already in a different code base rfr Pull request is ready for review

Development

Successfully merging this pull request may close these issues.

1 participant