D6852: Reset last_active_client when a ShellClient is removed

David Edmundson noreply at phabricator.kde.org
Sun Jul 23 14:41:48 UTC 2017


davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.


  there's 3 other differences between removeClient that I think should also be here:
  
  ---
  
    should_get_focus.removeAll(c);
  
  (this looks like it could lead to a similar crash)
  
  ---
  
    if (c == most_recently_raised)
        most_recently_raised = 0;
  
  ---
  
  #ifdef KWIN_BUILD_TABBOX
  
    if (tabBox->isDisplayed())
        tabBox->reset(true);
  
  #endif

REPOSITORY
  R108 KWin

BRANCH
  fix-last-active-client-wayland

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

To: graesslin, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170723/f25e9eea/attachment-0001.html>


More information about the Plasma-devel mailing list