On Tue, 15 Apr 2025 05:19:38 GMT, Tejesh R <t...@openjdk.org> wrote:

> Open source these AWT ScrollPane tests:
> 
> java/awt/ScrollPane/ScrollPaneSize/ScrollPaneSize.java
> java/awt/ScrollPane/ScrollPaneAsNeededTest/ScrollPaneAsNeededTest.java
> java/awt/ScrollPane/ScrollPaneComponentTest/ScrollPaneComponentTest.java
> java/awt/ScrollPane/ScrollPaneScrollType/ScrollPaneEventType.java
> java/awt/ScrollPane/ScrollPanechildViewportTest/ScrollPanechildViewportTest.java

test/jdk/java/awt/ScrollPane/ScrollPaneSize.java line 97:

> 95:         add(b3 = new Button("Button3"));
> 96: 
> 97:         setBackground(Color.white);

background color set twice?

test/jdk/java/awt/ScrollPane/ScrollPanechildViewportTest.java line 129:

> 127:         public Dimension getPreferredSize() {
> 128:             Dimension d = null;
> 129:             Dimension pd = ((ScrollPane) getParent()).getViewportSize();

use switch expression (instead of switch statement) to assign a value to d?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24647#discussion_r2045294917
PR Review Comment: https://git.openjdk.org/jdk/pull/24647#discussion_r2045290720

Reply via email to