D8917: Reduce the amount of spurious property changes on ColorScope

[Aleix Pol Gonzalez]

apol created this revision.
apol added reviewers: Plasma, mart.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  At the moment whenever something changed we were emitting colorGroupChanged
  and then every color would recompute. This would end up being emitted
  over 10 times at plasma startup so far.
  This patch makes sure that the property will only be emitted if the color
  group actually changes.

TEST PLAN
  Ran plasma, didn't notice issues.
  I don't see all of the changes on the property anymore

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  master

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

AFFECTED FILES
  src/declarativeimports/core/colorscope.cpp
  src/declarativeimports/core/colorscope.h

To: apol, #plasma, mart
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171120/36b9c5c1/attachment.html>