RFR: 8352149: Test java/awt/Frame/MultiScreenTest.java fails: Window list is empty [v2]

[Alexey Ivanov]

On Wed, 30 Apr 2025 17:21:21 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>>> #21942
>> 
>> Thanks @aivanov-jdk for suggesting the correct route.
>> 
>> @kboulanou Please modify the test to run for all platforms and you can limit the number of graphics configuration to 10 for linux.
>
>> @kumarabhi006 I am aware that I should not force push to PR. But, I made the mistake of pushing updates (sync) from master to this PR so it added 200 commits to this PR, which will make the PR confusing. I should have updated master and fork and rebased my branch on it and then push to PR. I will be more careful in the future. Please suggest if this is OK. Thanks.
> 
> @kboulanou Merging master is safe.
> 
> **Don't use `git rebase`**, though. Never use `git rebase` after you published your branch for PR.
> 
> If you merge (sync) changes from master, Git won't show them as differences. The PR compares your branch to master. By merging, you add more commits that already exist in master, which means no changes between master and your branch are added.

> @aivanov-jdk May I go ahead and issue integrate command to this PR ?

Yes. I'll leave the PR open for a while before I sponsor it to give anyone else to look at the updated code.

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

PR Comment: https://git.openjdk.org/jdk/pull/24752#issuecomment-2948738191