RFR: 8255241: [TestBug] Re-enable few ignored tests in javafx.controls module that pass with latest code

Kevin Rushforth kcr at openjdk.java.net
Thu Oct 22 15:40:11 UTC 2020


On Thu, 22 Oct 2020 12:13:11 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

> Some of the ignored unit tests in javafx.controls module pass with latest code.
> This bug is to address -
> - removal of ignored tag
> - fixing the test if fix is minor

It works for me on Windows. Have you run it on more than one platform?

I left one suggestion for you regarding the constants in TableColumnShim.

modules/javafx.controls/src/shims/java/javafx/scene/control/TableColumnShim.java line 34:

> 32: 
> 33:     // NOTE: These constants are direct copy of TableColumnBase
> 34:     public static final double DEFAULT_WIDTH = 80.0F;

Can you define these to reference those constants rather than copying them, like this?

     public static final double DEFAULT_WIDTH = TableColumnBase.DEFAULT_WIDTH ;

If so, you can remove the comments about needing to keep them in sync.

modules/javafx.controls/src/main/java/javafx/scene/control/TableColumnBase.java line 121:

> 119: 
> 120:     // NOTE: If these numbers change, update the copy of this value in
> 121:     // TableColumnHeader and in TableColumnShim

I think you can revert this change.

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

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


More information about the openjfx-dev mailing list