move the null check under if(added)

Author: romtsn

sentry-android-replay/src/main/java/io/sentry/android/replay/WindowRecorder.kt

@@ -108,12 +108,13 @@ internal class WindowRecorder(
   }
 
   override fun onRootViewsChanged(root: View, added: Boolean) {
-    if (root.phoneWindow == null) {
-      options.logger.log(WARNING, "Root view does not have a phone window, skipping.")
-      return
-    }
     rootViewsLock.acquire().use {
       if (added) {
+        if (root.phoneWindow == null) {
+          options.logger.log(WARNING, "Root view does not have a phone window, skipping.")
+          return
+        }
+
         rootViews.add(WeakReference(root))
         capturer?.recorder?.bind(root)
         determineWindowSize(root)

sentry-samples/sentry-samples-android/build.gradle.kts

@@ -121,6 +121,8 @@ dependencies {
     kotlin(Config.kotlinStdLib, org.jetbrains.kotlin.config.KotlinCompilerVersion.VERSION)
   )
 
+  //noinspection UseTomlInstead
+  implementation("com.smartlook.android:smartlook-analytics:2.3.2")
   implementation(projects.sentryAndroid)
   implementation(projects.sentryAndroidFragment)
   implementation(projects.sentryAndroidTimber)

sentry-samples/sentry-samples-android/src/main/AndroidManifest.xml

@@ -101,7 +101,7 @@
         <!--    <meta-data android:name="io.sentry.anr.enable" android:value="false" />-->
 
         <!--    how to disable the auto-init-->
-        <!--    <meta-data android:name="io.sentry.auto-init" android:value="false" />-->
+            <meta-data android:name="io.sentry.auto-init" android:value="false" />
 
         <!--    how to disable the NDK-->
         <!--    <meta-data android:name="io.sentry.ndk.enable" android:value="false" />-->
@@ -129,11 +129,11 @@
 <!--        <meta-data android:name="io.sentry.traces.profiling.sample-rate" android:value="1.0" />-->
 
         <!-- Enable profiling, adjust in production env -->
-        <meta-data android:name="io.sentry.traces.profiling.session-sample-rate" android:value="1.0" />
+        <meta-data android:name="io.sentry.traces.profiling.session-sample-rate" android:value="0.0" />
         <!-- Set profiling lifecycle, can be `manual` or `trace` -->
         <meta-data android:name="io.sentry.traces.profiling.lifecycle" android:value="manual" />
         <!-- Enable profiling on app start -->
-        <meta-data android:name="io.sentry.traces.profiling.start-on-app-start" android:value="true" />
+        <meta-data android:name="io.sentry.traces.profiling.start-on-app-start" android:value="false" />
 
         <!--    how to disable the Activity auto instrumentation for tracing-->
         <!--    <meta-data android:name="io.sentry.traces.activity.enable" android:value="false" />-->

sentry-samples/sentry-samples-android/src/main/java/io/sentry/samples/android/MyApplication.java

@@ -3,6 +3,7 @@
 import android.app.Application;
 import android.os.StrictMode;
 import io.sentry.Sentry;
+import io.sentry.android.core.SentryAndroid;
 
 /** Apps. main Application. */
 public class MyApplication extends Application {
@@ -13,8 +14,12 @@ public void onCreate() {
     strictMode();
     super.onCreate();
 
+    // Smartlook.getInstance().getPreferences().setProjectKey("528377853758d75ddf24c84b801366e4dc6bf2d1");
+    // Smartlook.getInstance().getPreferences().setRenderingMode(RenderingMode.WIREFRAME);
+    // Smartlook.getInstance().start();
     // Example how to initialize the SDK manually which allows access to SentryOptions callbacks.
     // Make sure you disable the auto init via manifest meta-data: io.sentry.auto-init=false
+    SentryAndroid.init(this);
     // SentryAndroid.init(
     //    this,
     //    options -> {

settings.gradle.kts

@@ -12,6 +12,9 @@ dependencyResolutionManagement {
         google()
         mavenCentral()
         mavenLocal()
+        maven {
+            url = uri("https://sdk.smartlook.com/android/release")
+        }
     }
 }