8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64

[amosshi]

Backport of JDK-8225045

• This PR contains two commits
• commit 1 is generated by git patch command, is a clean backport for JInternalFrameIconTest.java
• commit 2 is the manual merge of test/jdk/ProblemList.txt.rej file. The content of the file is bellow.

@@ -751,7 +751,6 @@
 javax/swing/JComboBox/8182031/ComboPopupTest.java 8196465 linux-all,macosx-all
 javax/swing/JFileChooser/6738668/bug6738668.java 8194946 generic-all
 javax/swing/JInternalFrame/Test6325652.java 8224977 macosx-all
-javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java 8225045 linux-all
 javax/swing/JPopupMenu/4870644/bug4870644.java 8194130 macosx-all,linux-all
 javax/swing/PopupFactory/6276087/NonOpaquePopupMenuTest.java 8065099,8208565 macosx-all,linux-all
 javax/swing/UIDefaults/6302464/bug6302464.java 8199079 macosx-all

Testing

• Local on MacOS: Test passed on MacOS 14.6.1 on Apple M1 Max
  • JInternalFrameIconTest.java: Test results: passed: 1
• Local on Linux: Test passed on Ubuntu 22.04.4 by @Harry-Junhua-Huang
  • Passed: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java
• Pipeline: All passed except macOS
  • macOS: /Applications/Xcode_14.3.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/stdio.h:188:1: note: 'sprintf' has been explicitly marked deprecated here The issue exists in all recent PR in jdk11u-dev and not caused by Current PR
• Testing Machine: SAP nightlies SKIPPED on 2024-08-23
  • Automated jtreg test: jtreg_jdk_tier4, Started at 2024-08-22 21:03:43+01:00
  • javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java: SKIPPED [Filter: Keywords - Not matching the given keyword expression: (((!headful)&(!printer)&(!intermittent))) & !manual & !ignore] GitHub 📊 - [0 msec]

---

Progress

• [x] Change must not contain extraneous whitespace
• [x] Commit message must refer to an issue
• [ ] JDK-8225045 needs maintainer approval

Issue

• JDK-8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 (Bug - P4 - Requested)

Reviewing

Using git

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

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

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2918

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

Using diff file

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

Webrev

Link to Webrev Comment

[bridgekeeper[bot]]

:wave: Welcome back ashi! 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[bot]]

@amosshi This change now passes all automated pre-integration checks.

After integration, the commit message for the final commit will be:

8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 17 new commits pushed to the master branch:

• dd35f187fd038281ce36b986ef71d3b813c0a83e: 8340671: GHA: Bump macOS and Xcode versions to macos-12 and XCode 13.4.1
• b2d385978fca9eb2fd2eb260a72d727d5f0d6740: Merge
• 0358cbd0a1538937612e3a78ffbfb64d7f4c45a2: 8296410: HttpClient throws java.io.IOException: no statuscode in response for HTTP2
• 934decc1475e5acc00be23c87507274f005780b4: Merge
• 90ad5b18de314faca19e322bb21a4c33cec54785: 8337664: Distrust TLS server certificates issued after Oct 2024 and anchored by Entrust Root CAs
• 6ee8bacfdda51b53178913ebf711ac9feacce26e: 8338139: {ClassLoading,Memory}MXBean::isVerbose methods are inconsistent with their setVerbose methods
• ffcdc104058ab2b58d8303f318c2e0ee0dc93a54: 8315936: Parallelize gc/stress/TestStressG1Humongous.java test
• 76b8327ba08e2d57b64cd266a3eb3f1a3232a1a5: 8328642: Convert applet test MouseDraggedOutCauseScrollingTest.html to main
• 8c31358ccc9d07f9512bc56159c85ce465e25135: 8328300: Convert PrintDialogsTest.java from Applet to main program
• ce572dfede792da0eb1b98ac3d8d1df7064dea33: 8339082: Bump update version for OpenJDK: jdk-11.0.26
• ... and 7 more: https://git.openjdk.org/jdk11u-dev/compare/cd9e6cb584bfeb232bcea7a642664cb6dd4424f3...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

[openjdk[bot]]

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

[openjdk[bot]]

⚠️ @amosshi This change is now ready for you to apply for maintainer approval. This can be done directly in each associated issue or by using the /approval command.

[mlbridge[bot]]

Webrevs

• 00: Full (af4173e6)

[mrserb]

This bug was reported on linux so you need to test the backport on linux as well.

[Harry-Junhua-Huang]

Test on local linux machine

System info

• tail /etc/lsb-release

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.4 LTS"

• lscpu

Architecture:            x86_64
  CPU op-mode(s):        32-bit, 64-bit
  Address sizes:         48 bits physical, 48 bits virtual
  Byte Order:            Little Endian
CPU(s):                  16
  On-line CPU(s) list:   0-15
Vendor ID:               AuthenticAMD
  Model name:            AMD Ryzen 7 5800H with Radeon Graphics
    CPU family:          25
    Model:               80
    Thread(s) per core:  2
    Core(s) per socket:  8
    Socket(s):           1
    Stepping:            0
    BogoMIPS:            6387.77

Test result: passed

• Passed: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java

[amosshi]

/approval request "Backporting for parity with 11.0.25-oracle. No risk, test fix. Clean backport. Test passes in local on both Ubuntu Linux and MacOS”

[openjdk[bot]]

@amosshi 8225045: The approval request has been created successfully.

[bridgekeeper[bot]]

@amosshi 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 add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

[bridgekeeper[bot]]

@amosshi 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 add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

[bridgekeeper[bot]]

@amosshi 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 add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

[bridgekeeper[bot]]

@amosshi 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 add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

[amosshi]

/integrate

[openjdk[bot]]

Going to push as commit 249144c02be9d0e51bef1b3a88bd8eeadb5a8d2f. Since your change was applied there have been 48 commits pushed to the master branch:

• 04e6f37d2c3b0983db92184f49391b4453b459df: 8247706: Unintentional use of new Date(year...) with absolute year
• add90003d601eec60cf9166f6013f0e86382729d: 8232367: Update Reactive Streams to 1.0.3 -- tests only
• 827528c3228705496813887f92f964310b010464: 8339470: [17u] More defensive fix for 8163921
• 96805ae97b5153b48d5fa5d1dac99a3763f9cde9: 8224624: Inefficiencies in CodeStrings::add_comment cause timeouts
• 98161b7a9b1ff4a9f3b6c41efd15c31a8f959c68: 8334332: TestIOException.java fails if run by root
• 9bf55020f074e77df170d7f5bbf807db3a183232: 8342426: [11u] javax/naming/module/RunBasic.java javac compile fails
• 8ba0341757545856666fbfcd3e7f9d715f5c0f86: Merge
• cee8535a9d3de8558b4b5028d68e397e508bef71: 8341675: [11u] Remove designator DEFAULT_PROMOTED_VERSION_PRE=ea for release 11.0.25
• 86d5188b8939c05abc2340c4c389c5fca640c485: 8332644: Improve graph optimizations
• 34a77a471b260f3b6f65c6cc61609d51eeade193: 8290367: Update default value and extend the scope of com.sun.jndi.ldap.object.trustSerialData system property
• ... and 38 more: https://git.openjdk.org/jdk11u-dev/compare/cd9e6cb584bfeb232bcea7a642664cb6dd4424f3...master

Your commit was automatically rebased without conflicts.

[openjdk[bot]]

@amosshi Pushed as commit 249144c02be9d0e51bef1b3a88bd8eeadb5a8d2f.

:bulb: You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.