On 4/8/2014 9:19 PM, Vivi An wrote:
Hello,
Could you please review the fix for JDK 9?
This bug is JAB related. ActivateDescenderPropertyChanged event for
JTree and JTable were not sent properly in case SHIFT+CursorDown or
Ctrl+CursorUp/Down are used. Fix made mainly uses lead path (acctive
child path) instead of selection path to check if an event needs to be
fired.
Bug: https://bugs.openjdk.java.net/browse/JDK-8036983
Webrev: http://cr.openjdk.java.net/~dmarkov/8036983/jdk9/webrev.00/
+ public void fireActiveDescendantPropertyChange(TreePath
oldPath, TreePath newPath) {
Is it possible to make the method package access instead of public?
+ int focusedRow =
JTable.this.getSelectionModel().getLeadSelectionIndex();;
There is one more semicolon at the end.
Is it possible to write a test (manual or automated) for the fix?
Thanks,
Alexandr.
Thanks
~ Vivi
