Integrated: 8255679: RegionBackgroundImageUITest.unalignedImage_Cover fails because of wrong color

[Jayathirth D V]

On Wed, 13 Mar 2024 08:56:13 GMT, Jayathirth D V <jdv at openjdk.org> wrote:

> Updated test to get screen capture in different platforms to see how we are picking colors. Found that this test fails because it assumes that red,green & blue bands in the test image are of equal size and picks color based on that assumption.
> 
> If we take 20 * 20 aligned image, each scanline is 3 red pixels -> 3 green pixels -> 8 blue pixels -> 3 green pixels -> 3 red pixels. So the blue pixels actually acquire more space in a scanline. Even in y direction pattern is same.
> 
> If we 48 * 48 unaligned image, each scanline is 7 red pixels -> 8 green pixels -> 19 blue pixels -> 7 green pixels -> 7 red pixels. So the blue pixels actually acquire more space in a scanline. Even in y direction pattern is same.
> 
> So when this 48 * 48 image is scaled to 336 * 336 in case of unalignedImage_Cover, we are calculating picking point for green color in such a way that it is hitting border of green and blue bands and test is failing. Only in this specific scenario we are scaling by large amount and the inappropriate color picking logic is failing.
> 
> Updated color picking logic in such a way that appropriate weightage is given to each color and we pick that color properly even when we have large scaling. With this change test is passing on all platforms and there are no regressions seen in all test run.

This pull request has now been integrated.

Changeset: c196454e
Author:    Jayathirth D V <jdv at openjdk.org>
Committer: Ajit Ghaisas <aghaisas at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/c196454e76dec1cbcd028e12d99f052981a63a4e
Stats:     7 lines in 1 file changed: 1 ins; 1 del; 5 mod

8255679: RegionBackgroundImageUITest.unalignedImage_Cover fails because of wrong color

Reviewed-by: aghaisas

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

PR: https://git.openjdk.org/jfx/pull/1400