<table><tr><td style="">subdiff 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/D1230" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D1230#158816" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D1230#158816</a>, <a href="https://phabricator.kde.org/p/Kanedias/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@Kanedias</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Well, it has <tt style="background: #ebebeb; font-size: 13px;">active</tt> property in KWayland that is true only if at least one client is connected (if I remember my code from 6 months back correctly), so I can expose it and use here. Will this suffice?</p></div>
</blockquote>

<p>I think you do already what I meant with the <tt style="background: #ebebeb; font-size: 13px;">isBound()</tt> call. Sorry for the confusion. :)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Then in general did you look at GNOME's way forward of screen recording/sharing on Wayland?</p></blockquote>

<p>No, sorry. This patchset was ready long before the GNOME news. I use different idea here that was proposed by Martin Graesslin, with GBM and Wayland protocol we actually can pass file descriptors to video buffer objects from one app to another <br />
 so KWin is passing BOs to KRfb which then can do all heavy-lifting without hurting KWin performance or stability.</p></blockquote>

<p>The thing is I would like to see your solution also be applicable to other apps that need access to the buffer contents, like screen recording tools (SimpleScreenRecorder for example) and so on. If you say KRfb does the heavy-lifting, I assume such a screen recording tool would need to do this as well in a nontrivial additional backend.</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/D1230" rel="noreferrer">https://phabricator.kde.org/D1230</a></div></div><br /><div><strong>To: </strong>Kanedias, graesslin, davidedmundson, subdiff, KWin<br /><strong>Cc: </strong>subdiff, ngraham, alexeymin, aacid, kwin, KWin, davidedmundson, plasma-devel, bwowk, leezu, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein<br /></div>