D9196: Don't set a window icon for plasmashell
    Eike Hein 
    noreply at phabricator.kde.org
       
    Tue Dec  5 11:42:24 UTC 2017
    
    
  
hein created this revision.
hein added reviewers: Plasma, davidedmundson, mart.
Restricted Application added a project: Plasma.
REVISION SUMMARY
  Setting a window icon is surprisingly costly and a decent chunk of
  startup time. The shell doesn't need one. The only place I could
  find where this icon is visible is in the KSysguard process list :).
  
  Some people may be reminded of this rejected request:
  
  https://git.reviewboard.kde.org/r/128484/
  
  But that's Plasma::Dialog in plasma-framework, which (currently)
  still sets an icon. This is about the shell.
REPOSITORY
  R120 Plasma Workspace
BRANCH
  master
REVISION DETAIL
  https://phabricator.kde.org/D9196
AFFECTED FILES
  shell/main.cpp
To: hein, #plasma, davidedmundson, mart
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171205/8b3c23b0/attachment-0001.html>
    
    
More information about the Plasma-devel
mailing list