Address PR comments

Author: markushi

sentry/src/main/java/io/sentry/SentryReplayOptions.java

@@ -281,11 +281,11 @@ public void setSessionSampleRate(final @Nullable Double sessionSampleRate) {
    * <p>Default is enabled.
    */
   public void setMaskAllText(final boolean maskAllText) {
-    trackCustomMasking();
     if (maskAllText) {
       maskViewClasses.add(TEXT_VIEW_CLASS_NAME);
       unmaskViewClasses.remove(TEXT_VIEW_CLASS_NAME);
     } else {
+      trackCustomMasking();
       unmaskViewClasses.add(TEXT_VIEW_CLASS_NAME);
       maskViewClasses.remove(TEXT_VIEW_CLASS_NAME);
     }
@@ -300,11 +300,11 @@ public void setMaskAllText(final boolean maskAllText) {
    * <p>Default is enabled.
    */
   public void setMaskAllImages(final boolean maskAllImages) {
-    trackCustomMasking();
     if (maskAllImages) {
       maskViewClasses.add(IMAGE_VIEW_CLASS_NAME);
       unmaskViewClasses.remove(IMAGE_VIEW_CLASS_NAME);
     } else {
+      trackCustomMasking();
       unmaskViewClasses.add(IMAGE_VIEW_CLASS_NAME);
       maskViewClasses.remove(IMAGE_VIEW_CLASS_NAME);
     }

sentry/src/test/java/io/sentry/SentryReplayOptionsTest.kt

@@ -177,10 +177,18 @@ class SentryReplayOptionsTest {
   }
 
   @Test
-  fun `trackCustomMasking only adds integration once`() {
+  fun `setMaskAllText true does not set custom integration`() {
     val options = SentryReplayOptions(false, null)
     options.setMaskAllText(true)
     options.setMaskAllImages(true)
+    assertFalse(hasCustomMaskingIntegration())
+  }
+
+  @Test
+  fun `trackCustomMasking only adds integration once`() {
+    val options = SentryReplayOptions(false, null)
+    options.setMaskAllText(false)
+    options.setMaskAllImages(false)
     assertTrue(hasCustomMaskingIntegration())
     assertEquals(
       1,
@@ -189,7 +197,6 @@ class SentryReplayOptionsTest {
       },
     )
   }
-
   @Test
   fun `addMaskViewClass adds ReplayCustomMasking integration`() {
     val options = SentryReplayOptions(false, null)