QQuickComboBox: fix crash on popup destructionv5.9.0-rc2v5.9.0-rc1v5.9.0

f19e2d5f changed the QQuickComboBox destructor to call setPopup(null)
instead of deleting the popup directly. The reason for this change was
that setPopup() did not only destroy the popup but also disconnected
the visibleChanged() signal to avoid spurious highlightedIndexChanged()
signal emission during ComboBox destruction. There was an undesired
side-effect with this change: it ended up calling destroyDelegate()
during destruction, potentially accessing a destroyed QML context.
Switch back to the old way, and disconnect visibleChanged() in place
instead.

Task-number: QTBUG-57650
Task-number: QTBUG-50992
Change-Id: I49d96710815dfc163e75a7ff82fe0e4742cfbb59
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>

0 files changed, 0 insertions, 0 deletions