tree 059862eea6061af214e92879dd44890c0571fbfe
parent 6b6c9528d7eddea90f7b76014bf075ae69cb88f6
author Patrick Tasse <patrick.tasse@gmail.com> 1603824741 -0400
committer Patrick Tasse <patrick.tasse@gmail.com> 1603986034 -0400

tmf: Fix obsolete checked elements in AbstractSelectTreeViewer/2

AbstractSelectTreeViewer/2 uses a checkbox tree that internally stores
its checked elements.

When the tree model is updated from the data provider, the element
instances can change even if they retain the same model ids.

The checkbox tree needs to re-set and update its internal set of checked
elements. This happens in contentChanged() which is now only called when
the tree input has changed, since https://git.eclipse.org/r/169994.

This patch changes AbstractTmfTreeViewer to always call contentChanged()
when updating the tree, whether the tree input has changed or not.

[Fixed] Fix obsolete checked elements in AbstractSelectTreeViewer/2

Change-Id: I5ec79c12000d7ab8cfcb81c02a77ddb3aa24648b
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/171377
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>