D17097: [scripting] Emit clientAdded for Wayland clients

zzag reclaimed this revision.
zzag added a comment.


  In D17097#365086 <https://phabricator.kde.org/D17097#365086>, @graesslin wrote:
  
  > The problem with the approach is that we cannot secure the kwinrc to which it is written. So the whole thing doesn't really work as long as all processes can write kwinrc. On the other hand it could protect properly sandboxed flatpacks.
  
  
  Could we use kwallet for such a purpose? Also, do web browsers try to secure the list of enabled extensions?
  
  > Btw. I don't think we need to discard the reviews. I could also see a solution in disabling the dbus extension on Wayland or to say we ignore the security problems due to the point written above.
  
  Oh, I probably misunderstood the second last comment.
  
  Also, I'd like to point out that the windowing system is already "open":
  
    var clients = [];
    
    workspace.clientActivated.connect(function (client) {
        if (!client) {
            return;
        }
    
        if (clients.indexOf(client) != -1) {
            return;
        }
    
        clients.push(client);
    });
    
    workspace.clientRemoved.connect(function (client) {
        var index = clients.indexOf(client);
        if (index == -1) {
            return;
        }
    
        clients.splice(index, 1);
    });

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D17097

To: zzag, #kwin
Cc: mart, graesslin, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol