On Fri, 15 Aug 2025 17:29:16 GMT, Khalid Boulanouare <d...@openjdk.org> wrote:

>> This PR will consolidate fixes of the following bugs:
>> 
>> https://bugs.openjdk.org/browse/JDK-8361188
>> https://bugs.openjdk.org/browse/JDK-8361189
>> https://bugs.openjdk.org/browse/JDK-8361190
>> https://bugs.openjdk.org/browse/JDK-8361191
>> https://bugs.openjdk.org/browse/JDK-8361192
>> https://bugs.openjdk.org/browse/JDK-8361193
>> https://bugs.openjdk.org/browse/JDK-8361195
>> 
>> This PR depends on https://github.com/openjdk/jdk/pull/25971
>> 
>> For test : java/awt/Mixing/AWT_Mixing/JComboBoxOverlapping.java, the fix 
>> suggested is to return false in method isValidForPixelCheck for embedded 
>> frame, in which case the component is set to null. For more details see bug: 
>> [JDK-8361188](https://bugs.openjdk.org/browse/JDK-8361188)
>> 
>> For test : test/jdk/java/awt/Mixing/AWT_Mixing/MixingPanelsResizing.java, I 
>> had to create a a tolerance color matching method for mac for the tests to 
>> pass. Also, the jbuttons needed to have different color than the color of 
>> the background frame, in order for test to pass. For more detail see bug: 
>> https://bugs.openjdk.org/browse/JDK-8361193
>> 
>> For test : test/jdk/java/awt/Mixing/AWT_Mixing/JSplitPaneOverlapping.java, 
>> it seems that color selected for lightweight component matches the 
>> background color of the frame. And this will cause the test to fail when 
>> matching colors. Choosing any color different than the background color will 
>> get the test to pass. For more details, see bug: 
>> https://bugs.openjdk.org/browse/JDK-8361192
>> 
>> For test test/jdk/java/awt/Mixing/AWT_Mixing/JPopupMenuOverlapping.java, it 
>> looks like the frame when visible, the popup test does not work properly. 
>> The frame needs to be hidden for the test to click on popup. For more 
>> details see bug: https://bugs.openjdk.org/browse/JDK-8361191
>> 
>> For test test/jdk/java/awt/Mixing/AWT_Mixing/JMenuBarOverlapping.java, the 
>> test runs successfully but it times out after the default 2 minutes of 
>> jtreg. increasing the timeout to 3 minutes get the test to pass. For more 
>> details please refer to bug: https://bugs.openjdk.org/browse/JDK-8361190
>
> Khalid Boulanouare has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Removes extra white lines

Modifications tested and passed.

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

PR Comment: https://git.openjdk.org/jdk/pull/26625#issuecomment-3195587560

Reply via email to