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