D21172: FrameSvg: also clear mask cache on clearCache()

2019-05-14 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:f008615e3710: FrameSvg: also clear mask cache on 
clearCache() (authored by kossebau).

REPOSITORY
  R242 Plasma Framework (Library)

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D21172?vs=57960=58087

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

AFFECTED FILES
  src/plasma/framesvg.cpp

To: kossebau, #plasma, davidedmundson
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D21172: FrameSvg: also clear mask cache on clearCache()

2019-05-13 Thread David Edmundson
davidedmundson accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  clearalsomaskcache

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

To: kossebau, #plasma, davidedmundson
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D21172: FrameSvg: also clear mask cache on clearCache()

2019-05-12 Thread Friedrich W. H. Kossebau
kossebau created this revision.
kossebau added a reviewer: Plasma.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
kossebau requested review of this revision.

REVISION SUMMARY
  It can be unexpected that those are not reset when clearCache() was called.
  
  With the current logic when a theme changes the mask cache not reset also
  results in wrong lookups, as the cacheid used as key does not include the
  theme name.

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  clearalsomaskcache

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

AFFECTED FILES
  src/plasma/framesvg.cpp

To: kossebau, #plasma
Cc: kde-frameworks-devel, michaelh, ngraham, bruns