<table><tr><td style="">dfaure created this revision.<br />dfaure added a reviewer: mlaurent.<br />dfaure added a subscriber: KDE PIM.<br />Restricted Application added a project: KDE PIM.<br />Restricted Application added a subscriber: kde-pim.</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/D2637" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This call to load() triggers a lot of internal stuff in WebEngine,<br />
including NavigateToEntry, creation (and soon afterwards, deletion) of<br />
QtWebEngineCore::RenderWidgetHostViewQtDelegateWidget, etc.</p>

<p>I can't see any regression when removing this call.</p>

<p>On the contrary, this fixes a focus issue: when clicking on an<br />
external link in an email and coming back to kmail, arrow up/down<br />
didn't work anymore, because the last widget installed as focusproxy<br />
for the webview (the DelegateWidget above) had been deleted meanwhile.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rMESSAGELIB PIM: Message Library</div></div></div><br /><div><strong>BRANCH</strong><div><div>Applications/16.08</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2637" rel="noreferrer">https://phabricator.kde.org/D2637</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>messageviewer/src/htmlwriter/webengineparthtmlwriter.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>dfaure, mlaurent<br /><strong>Cc: </strong>kde-pim, KDE PIM, spencerb, dvasin, winterz, smartins, vkrause, mlaurent, knauss, dvratil<br /></div>