Review Request 125711: Use deleteLater in Part::slotWidgetDestroyed().

[Alex Merry]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125711/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: kparts


Description
-------

When a part's main widget is destroyed, we should not delete the part
immediately - both because deleting an object from inside one of its
signals is not safe generally, and because derived classes may well hold
QPointers to the widgets, expecting them to be cleared between the
widget being destroyed and the part being destroyed.


Diffs
-----

  autotests/parttest.cpp c477face2f0f94f1d66deadedb4bc913eaa21842 
  src/part.cpp 4a3150e8aa2a6ed2aff68c945135b3536db2059d 

Diff: https://git.reviewboard.kde.org/r/125711/diff/


Testing
-------

Unit tests for KParts pass.

Running akregator under valgrind no longer produces a bunch of "invalid reads" from KHTMLPart when switching around between articles.


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151019/5db56bfc/attachment.html>