fvogt created this revision. Restricted Application added a project: Plasma.
REVISION SUMMARY While Qt's event loop checks for POLLHUP, it treats it exactly like POLLIN and it's not possible to retrieve that information from Qt. So in the Connection read handler it needs to be checked for again. TEST PLAN Before this patch plasma-browser-integration-host consumed 100% CPU after exiting the browser, 100% reproducible. Now it quits gracefully. REPOSITORY R856 Plasma Browser Integration BRANCH master REVISION DETAIL https://phabricator.kde.org/D6711 AFFECTED FILES host/connection.cpp To: fvogt, #plasma, broulik, davidedmundson Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas