On Tue, 7 Oct 2025 16:44:39 GMT, Tejesh R <[email protected]> wrote:

>>> > As @DamonGuy mentioned, a better way to deal with 
>>> > [JDK-8305915](https://bugs.openjdk.org/browse/JDK-8305915) could be 
>>> > closing it as _Cannot reproduce_ and submitting a new bug to update 
>>> > `FrameLocation.java` and remove `SizeMinimizedTest.java` from the problem 
>>> > list.
>>> 
>>> Probably I'll capture screenshot on failure and deproblemlist it. If I 
>>> close the bug as _Cannot reproduce_ then it remains in problem listing, so 
>>> better way is to de-problemlist it and add screen capture.
>> 
>> That's the point—you cannot reproduce the failure of `FrameLocation.java`, 
>> can you?
>> 
>> Thus, the correct resolution is *Cannot Reproduce*.
>> 
>> Adding a screenshot is not fixing the failure reported in 
>> [JDK-8305915](https://bugs.openjdk.org/browse/JDK-8305915), it's an 
>> improvement for the test to have the more information to analyse the failure 
>> if it occurs again. This is why it's a new issue.
>> 
>> 8305915 has to be removed from the PL, submit a new bug, or rather create a 
>> subtask of JDK-8305915, to remove both `FrameLocation.java` and 
>> `SizeMinimizedTest.java` from the problem list.
>> 
>> I strongly believe this approach creates a clearer bug history in JBS and 
>> clearer changelog in the JDK. This also makes backporting easier as well as 
>> dealing with test failures in older releases.
>> 
>> Yes, submitting two new bugs creates more work but we're not in a rush here, 
>> and I think *it's worth the time*.
>> 
>> I can submit the bugs for you if you prefer so.
>
> @aivanov-jdk @DamonGuy I've raised https://github.com/openjdk/jdk/pull/27678 
> PR for de-problemlisting the main bug. Once this is integrated, I'll close 
> the main bug as "Not-Reproducible".

> @TejeshR13 You can still use this PR to implement adding screenshots, just 
> change the subject of the PR to associate it with the new bugid.

New bugid ? You want me to raise another JBS to handle adding screenshots ?

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

PR Comment: https://git.openjdk.org/jdk/pull/27366#issuecomment-3379430004

Reply via email to