D7250: Avoid some unnecessary theme content lookups

2017-08-11 Thread Fabian Vogt
fvogt closed this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D7250 To: fvogt, #plasma, mart Cc: #frameworks

D7250: Avoid some unnecessary theme content lookups

2017-08-11 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D7250 To: fvogt, #plasma, mart Cc: #frameworks

D7250: Avoid some unnecessary theme content lookups

2017-08-11 Thread Fabian Vogt
fvogt added a reviewer: Plasma. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D7250 To: fvogt, #plasma Cc: #frameworks

D7250: Avoid some unnecessary theme content lookups

2017-08-11 Thread Fabian Vogt
fvogt updated this revision to Diff 18004. fvogt added a comment. Forgot a check REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7250?vs=18003&id=18004 BRANCH master REVISION DETAIL https://phabricator.kde.org/D7250 AFFECTED FILES

D7250: Avoid some unnecessary theme content lookups

2017-08-11 Thread Fabian Vogt
fvogt created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY imagePath can be an absolute path into an iconTheme -> Do not try to find it in the Plasma theme imagePath can be empty -> Do not try