<table><tr><td style="">marten updated this revision to Diff 53415.<br />marten retitled this revision from "Reassign ownership of QWebEngineProfile in MailWebEnginePage, to avoid crash on KMail exit or viewer window close" to "Move management of QWebEngineProfile to WebEnginePage, to avoid crash on KMail exit or viewer window close".<br />marten edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-n6q3sbcindab4cf/">(Show Details)</a><br />marten added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19559">View Revision</a></tr></table><br /><div><div><p>Setup and destruction of the QWebEngineProfile moved to WebEnginePage, so MailWebEngineView need know nothing about the profile and MailWebEnginePage only needs to access it via profile().</p>

<p>Akregator currently constructs  a WebEnginePage using its own profile, which however is always the default profile.  This works, because all web views with Akregator use the same browser settings, but changing Akregator and eliminating the 2-argument WebEnginePage constructor would mean that releases of Akregator and messagelib would have to be synchronised exactly.  For the moment I've just marked the 2-argument constructor as deprecated, and Akregator can be updated at a later date.</p></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Built messagelib with this change, observed no crash when quitting KMail or closing a message viewer window.  Without this change, the crash happens every time.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
Build Akregator using the updated messagelib, observed correct operation and display of articles and pages.</span><br />
</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D19559?vs=53357&id=53415">https://phabricator.kde.org/D19559?vs=53357&id=53415</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19559">https://phabricator.kde.org/D19559</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>messageviewer/src/viewer/webengine/mailwebenginepage.cpp<br />
messageviewer/src/viewer/webengine/mailwebenginepage.h<br />
messageviewer/src/viewer/webengine/mailwebengineview.cpp<br />
webengineviewer/src/webenginepage.cpp<br />
webengineviewer/src/webenginepage.h</div></div></div><br /><div><strong>To: </strong>marten, KDE PIM, dvratil<br /><strong>Cc: </strong>anthonyfieroni, dvratil, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss<br /></div>