<table><tr><td style="">zzag 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/D27788">View Revision</a></tr></table><br /><div><div><p>I still have concerns about the specified damage region. We only pass the region which must be repainted to repair the back buffer. We probably need to run prePaintScreen() and prePaintWindow() before calling eglSwapBuffersWithDamageEXT() in order to find out what region will be repainted in the current/next frame. Unfortunately, this would require some refactoring.</p>
<p>Please notice that effects such as blur may expand the painted area so we have to run prePaint hooks.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27788">https://phabricator.kde.org/D27788</a></div></div><br /><div><strong>To: </strong>apol, KWin, Plasma: Mobile<br /><strong>Cc: </strong>mwolff, zzag, davidedmundson, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>