[Approved] RFR: 8221334: TableViewSkin: must initialize flow's cellCount in constructor

Ajit Ghaisas aghaisas at openjdk.java.net
Fri Dec 6 06:54:31 UTC 2019


On Fri, 29 Nov 2019 15:08:16 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

> This is a fix for https://bugs.openjdk.java.net/browse/JDK-8221334
> 
> - fixed as outlined in the bug report: added updateItemCount() in skin constructor (that's what all sibling skins are doing)
> - added test which fails for TableViewSkin before, passes after the fix
> 
> ----------------
> 
> Commits:
>  - 55e588c7: 8221334: TableViewSkin: must initialize flow's cellCount in constructor
> 
> Changes: https://git.openjdk.java.net/jfx/pull/56/files
>  Webrev: https://webrevs.openjdk.java.net/jfx/56/webrev.00
>   Issue: https://bugs.openjdk.java.net/browse/JDK-8221334
>   Stats: 113 lines in 2 files changed: 113 ins; 0 del; 0 mod
>   Patch: https://git.openjdk.java.net/jfx/pull/56.diff
>   Fetch: git fetch https://git.openjdk.java.net/jfx pull/56/head:pull/56

Approved by aghaisas (Reviewer).

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


More information about the openjfx-dev mailing list