D19559: Reassign ownership of QWebEngineProfile in MailWebEnginePage, to avoid crash on KMail exit or viewer window close

Jonathan Marten noreply at phabricator.kde.org
Thu Mar 7 13:15:08 GMT 2019


marten added a comment.


  @dvratil:  in light of @anthonyfieroni's comments, should this fix go into WebEnginePage (messagelib/webengineviewer/src/webenginepage.cpp) instead?
  
  @anthonyfieroni:  are there any other applications within KDE PIM or elsewhere using WebEnginePage, so that I can build and test with them?  If this fix (latest diff 53357) were to be moved into WebEnginePage then it would remove the option for any user of that class to set their own QWebEngineProfile (because MailWebEngineView doesn't need to and will not).

REPOSITORY
  R94 PIM: Message Library

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

To: marten, #kde_pim, dvratil
Cc: anthonyfieroni, dvratil, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190307/c9855177/attachment.html>


More information about the kde-pim mailing list