Cold start measurement doesn't work in 7.12.1

[zhukic]

Integration

sentry-android

Build System

Gradle

AGP Version

8.5.1

Proguard

Enabled

Version

7.12.1

Steps to Reproduce

Updated SDK from 7.9.0 to 7.12.1

Performance V2 is enabled

Application launch from foreground(isForegroundImportance is true)

Expected Result

Cold start measurement works

Actual Result

Transactions are missing the app.start.cold span.

I debugged the SDK, and the problem is here

sentry-java/sentry-android-core/src/main/java/io/sentry/android/core/performance/AppStartMetrics.java

Line 237 in 7ee0493

public void registerApplicationForegroundCheck(final @NotNull Application application) {

The Runnable passed to Handler.post is called before AppStartMetrics.onActivityCreated so that appLaunchedInForeground set to false which leads to AppStartMetrics.getAppStartTimeSpanWithFallback returns the empty span.

[kahest]

@zhukic thanks for reporting - @stefanosiano pls take a look 🙏

[stefanosiano]

The Runnable passed to Handler.post is called before AppStartMetrics.onActivityCreated so that appLaunchedInForeground set to false which leads to AppStartMetrics.getAppStartTimeSpanWithFallback returns the empty span.

hey @zhukic
The runnable is put at the end of the main thread queue. If the app is starting and an Activity is going to be created, the creation of the Activity is already in the queue. This means that the runnable runs after the Activity onCreate().
I tried in our sample app and in a newly created project and it seems to work fine.
Could you provide a link to a transaction that should have a cold start span, but doesn't have one?