ScrollView doesn't use safe reference counting to hide indicators

[stevenbrix]

Describe the bug

We have a crash in our app, which is attributed to the ScrollView::OnHideIndicatorsTimerTick method that happens after a window has been closed. Looking at the implementation of how this event is registered, it is just grabbing a raw this pointer, so there is no guarantee the pointer is valid once the object has been destroyed.

C++/Winrt docs describing this issue: https://learn.microsoft.com/en-us/windows/uwp/cpp-and-winrt-apis/weak-references#the-issue

Steps to reproduce the bug

So timing related, good luck reproducing

Expected behavior

No response

Screenshots

No response

NuGet package version

None

Windows version

No response

Additional context

No response

[Hemantxk]

Hi @stevenbrix, can you share a heap dump for the crash if you have one handy? By the way, which application is running into this?

[Hemantxk]

Hi @stevenbrix , gentle ping on above request, can you share more details on issue as requested in my previous ping.
Thanks!

[stevenbrix]

@Hemantxk we don't have capture heap dumps to protect from capturing PII from our customers, but i can share a public link to our sentry crash. this is arc on windows