On Wed, 18 Aug 2021 15:58:21 GMT, Rahul Kumar 
<github.com+86982738+rahulk...@openjdk.org> wrote:

> Adding a new manual test case for accept attribute of input tag.
> This test is to verify the fix for 
> [JDK-8089915](https://bugs.openjdk.java.net/browse/JDK-8089915) .
> We have tested the test cases on all three platforms and its working as 
> expected.

The test looks good to me. I left two minor formatting comments.

tests/manual/web/InputTypeAcceptAttributeTest.java line 26:

> 24:  */
> 25: 
> 26: 

Minor: can you remove the extra blank line?

tests/manual/web/InputTypeAcceptAttributeTest.java line 37:

> 35: import javafx.stage.Stage;
> 36: import javafx.scene.layout.VBox;
> 37: import javafx.scene.layout.HBox;

Suggestion: maybe sort the list of imports?

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

PR: https://git.openjdk.java.net/jfx/pull/607

Reply via email to