<table><tr><td style="">davidedmundson requested changes to this revision.<br />davidedmundson added a comment.<br />This revision now requires changes to proceed.
</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/D26458">View Revision</a></tr></table><br /><div><div><p>One minor change, otherwise it's all good.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D26458#inline-151668">View Inline</a><span style="color: #4b4d51; font-weight: bold;">windowshadow.cpp:99</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">shadowManager</span><span style="color: #aa2211">-></span><span class="n">removeShadow</span><span class="p">(</span><span class="n">surface</span><span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">surface</span><span style="color: #aa2211">-></span><span class="n">commit</span><span class="p">(</span><span class="n">KWayland</span><span style="color: #aa2211">::</span><span class="n">Client</span><span style="color: #aa2211">::</span><span class="n">Surface</span><span style="color: #aa2211">::</span><span class="n">CommitFlag</span><span style="color: #aa2211">::</span><span class="n">None</span><span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">shadow</span> <span style="color: #aa2211">=</span> <span style="color: #aa4000">nullptr</span><span class="p">;</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Lets avoid this.</p>

<p style="padding: 0; margin: 8px;">Firstly it messes with Qt. We're doing this potentially during platform window initialisation, we're potentially committing before it's set some super important properties that we need set on the first commit.</p>

<p style="padding: 0; margin: 8px;">Secondly, we want it atomic with when the window updates (to a new theme or whatever) anyway.</p>

<p style="padding: 0; margin: 8px;">window->requestUpdate(); would be a compromise, it'll make Qt do a commit at the next available time.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R130 Frameworks integration plugin using KWayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26458">https://phabricator.kde.org/D26458</a></div></div><br /><div><strong>To: </strong>zzag, KWin, davidedmundson<br /><strong>Cc: </strong>davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>