[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    RFR: 8221334: TableViewSkin: must initialize flow's cellCount in constructor
From:       Jeanette Winzenburg <fastegal () openjdk ! java ! net>
Date:       2019-11-29 15:08:16
Message-ID: x0bZRO_ImjJesbinSBqEC9p2XAf1lpgw_C-pbhz3LDg=.7f29f5f5-4a6f-41c4-ba4f-daaf50f88be9 () github ! com
[Download RAW message or body]

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

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


[prev in list] [next in list] [prev in thread] [next in thread]