[Notifications] Fix crash accessing ViewTransition attached property

Authored by broulik on Dec 10 2019, 5:35 PM.



There's something really funky going on in this...
Avoid using ViewTransition attached property be just setting the animation target before triggering the model removal.

BUG: 414066

Test Plan
  • Remains to be seen whether the crash is gone but gdb showed it was related to QQuickViewTransitionAttached which we no longer use
  • Hasn't crashed for a while lol

Diff Detail

R120 Plasma Workspace
Lint Skipped
Unit Tests Skipped
broulik created this revision.Dec 10 2019, 5:35 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 10 2019, 5:35 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Dec 10 2019, 5:35 PM
broulik updated this revision to Diff 71219.
  • remove leftover debugging
davidedmundson accepted this revision.Dec 10 2019, 5:59 PM
This revision is now accepted and ready to land.Dec 10 2019, 5:59 PM

No more crashes so far. :)

This revision was automatically updated to reflect the committed changes.