D12527: Revert e88da8274468 in TextDocument to fix event handler issues

[Christoph Roick]

croick created this revision.
croick added a reviewer: KDevelop.
Restricted Application added a subscriber: kdevelop-devel.
croick requested review of this revision.

REVISION SUMMARY
  - Deleting the document directly may lead to calling the destructor again from the event handler during destruction (see D12131 <https://phabricator.kde.org/D12131>).
  - revert already attempted in 72e2bea870d6 <https://phabricator.kde.org/R33:72e2bea870d6762ba34923cef5b0b80388ff5326>, but never merged
  - `PartDocument::close` calls deleteLater on documents, so overriding the function is superfluous
  - The initial purpose (of fixing event handler issues) seems to be fulfilled already.

TEST PLAN
  - close KDevelop with a displayed kmarkdownwebview plugin

REPOSITORY
  R32 KDevelop

BRANCH
  textdocdelete

REVISION DETAIL
  https://phabricator.kde.org/D12527

AFFECTED FILES
  kdevplatform/shell/textdocument.cpp
  kdevplatform/shell/textdocument.h

To: croick, #kdevelop
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180425/afd54632/attachment.html>