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

2019-12-06 Thread Ajit Ghaisas
Changeset: 07af89a9 Author:Jeanette Winzenburg Committer: Ajit Ghaisas Date: 2019-12-06 11:25:18 + URL: https://git.openjdk.java.net/jfx/commit/07af89a9 8221334: TableViewSkin: must initialize flow's cellCount in constructor Reviewed-by: aghaisas ! modules/javafx.controls/s

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

2019-12-05 Thread Ajit Ghaisas
On Fri, 29 Nov 2019 15:08:16 GMT, Jeanette Winzenburg 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 Ta

Re: RFR: 8221334: TableViewSkin: must initialize flow's cellCount in constructor

2019-12-05 Thread Ajit Ghaisas
On Fri, 29 Nov 2019 15:08:16 GMT, Jeanette Winzenburg 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 Ta

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

2019-11-29 Thread Jeanette Winzenburg
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: - 55