Commit 76540577 authored by Ahmed Mehfooz's avatar Ahmed Mehfooz Committed by Commit Bot

Stop observing all animation sequences before deleting

StackedNotificationBarIcon.

Issue 100% reproducible without the fix.
With the fix the crash does not happen.
Confirmed over 3 runs for each case.

Bug: 1050685
Change-Id: Id3e5f7204f7f124a1d258b2f45864b451ff6647a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2055846Reviewed-by: default avatarTim Song <tengs@chromium.org>
Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#741586}
parent 3950ac54
...@@ -118,6 +118,7 @@ class StackedNotificationBar::StackedNotificationBarIcon ...@@ -118,6 +118,7 @@ class StackedNotificationBar::StackedNotificationBarIcon
} }
~StackedNotificationBarIcon() override { ~StackedNotificationBarIcon() override {
StopObserving();
if (is_animating_out()) if (is_animating_out())
layer()->GetAnimator()->StopAnimating(); layer()->GetAnimator()->StopAnimating();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment