On Wed, 20 Mar 2024 17:38:44 GMT, Phil Race <[email protected]> wrote:
>> A simple fix to the test to skip sampling the leftmost and rightmost ends of
>> the divider
>> to avoid possible issues at fractional scales.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8328540
Marked as reviewed by psadhukhan (Reviewer).
test/jdk/javax/swing/JSplitPane/4885629/bug4885629.java line 109:
> 107: for (int i = 1; i < rect.width - 1; i++) {
> 108: if (!BGCOLOR.equals(robot.getPixelColor(p.x + i,
> p.y + rect.height - 1))) {
> 109: throw new Error("The divider's area has
> incorrect color. i=" + i);
Should we still use "Error" or use RuntimeException for consistency with other
tests?
-------------
PR Review: https://git.openjdk.org/jdk/pull/18384#pullrequestreview-1950781009
PR Review Comment: https://git.openjdk.org/jdk/pull/18384#discussion_r1533262146