Address PR feedback

Author: markushi

sentry-android-core/src/main/java/io/sentry/android/core/ApplicationExitInfoEventProcessor.java

@@ -887,7 +887,7 @@ private void applyAnrProfile(
             new ExceptionMechanismException(mechanism, exception, null, false);
 
         final @NotNull List<SentryException> sentryException =
-            sentryExceptionFactory.getSentryExceptions(exception);
+            sentryExceptionFactory.getSentryExceptions(error);
 
         // Replace the original ANR exception with the profile-derived one,
         // as we assume the profiling culprit identification is more valuable

sentry-android-core/src/main/java/io/sentry/android/core/anr/AnrCulpritIdentifier.java

@@ -13,18 +13,22 @@
 public class AnrCulpritIdentifier {
 
   // common Java and Android packages who are less relevant for being the actual culprit
-  private static final List<String> systemAndFrameworkPackages = new ArrayList<>(9);
+  private static final List<String> systemAndFrameworkPackages = new ArrayList<>(11);
 
   static {
     systemAndFrameworkPackages.add("java.lang");
     systemAndFrameworkPackages.add("java.util");
+
     systemAndFrameworkPackages.add("android.app");
     systemAndFrameworkPackages.add("android.os.Handler");
     systemAndFrameworkPackages.add("android.os.Looper");
     systemAndFrameworkPackages.add("android.view");
     systemAndFrameworkPackages.add("android.widget");
     systemAndFrameworkPackages.add("com.android.internal");
     systemAndFrameworkPackages.add("com.google.android");
+
+    systemAndFrameworkPackages.add("kotlin");
+    systemAndFrameworkPackages.add("kotlinx.coroutines");
   }
 
   private static final class StackTraceKey {

sentry-android-core/src/main/java/io/sentry/android/core/anr/AnrProfilingIntegration.java

@@ -57,8 +57,8 @@ public void register(final @NotNull IScopes scopes, final @NotNull SentryOptions
             "SentryAndroidOptions is required");
     this.logger = options.getLogger();
 
-    if (((SentryAndroidOptions) options).isEnableAnrProfiling()) {
-      if (options.getCacheDirPath() == null) {
+    if (this.options.isEnableAnrProfiling()) {
+      if (this.options.getCacheDirPath() == null) {
         logger.log(SentryLevel.WARNING, "ANR Profiling is enabled but cacheDirPath is not set");
         return;
       }
@@ -74,12 +74,30 @@ public void close() throws IOException {
     enabled.set(false);
     AppState.getInstance().removeAppStateListener(this);
 
-    try (final @NotNull ISentryLifecycleToken ignored = profileManagerLock.acquire()) {
-      final @Nullable AnrProfileManager p = profileManager;
-      if (p != null) {
-        p.close();
+    final @Nullable SentryAndroidOptions opts = options;
+    if (opts != null) {
+      try {
+        opts.getExecutorService()
+            .submit(
+                new Runnable() {
+                  @Override
+                  public void run() {
+                    try (final @NotNull ISentryLifecycleToken ignored =
+                            profileManagerLock.acquire()) {
+                      if (profileManager != null) {
+                        //noinspection DataFlowIssue
+                        profileManager.close();
+                      }
+                    } catch (IOException e) {
+                      logger.log(SentryLevel.WARNING, "Failed to close AnrProfileManager");
+                    } finally {
+                      profileManager = null;
+                    }
+                  }
+                });
+      } catch (Throwable t) {
+        logger.log(SentryLevel.WARNING, "Failed to submit AnrProfileManager close");
       }
-      profileManager = null;
     }
   }
 
@@ -101,6 +119,7 @@ public void onForeground() {
       }
 
       final @NotNull Thread profilingThread = new Thread(this, "AnrProfilingIntegration");
+      profilingThread.setDaemon(true);
       profilingThread.start();
       thread = profilingThread;
     }