Remove reportFullDisplayed (#3717)

* replace synchronized with lock * Remove deprecated reportFullDisplayed
getsentry · Sep 30, 2024 · 6101b73 · 6101b73
1 parent f1b7116
commit 6101b73
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 37 deletions.
diff --git a/sentry/api/sentry.api b/sentry/api/sentry.api
@@ -901,7 +901,6 @@ public abstract interface class io/sentry/IScopes {
 	public abstract fun pushScope ()Lio/sentry/ISentryLifecycleToken;
 	public abstract fun removeExtra (Ljava/lang/String;)V
 	public abstract fun removeTag (Ljava/lang/String;)V
-	public fun reportFullDisplayed ()V
 	public abstract fun reportFullyDisplayed ()V
 	public abstract fun setActiveSpan (Lio/sentry/ISpan;)V
 	public abstract fun setExtra (Ljava/lang/String;Ljava/lang/String;)V
@@ -2370,7 +2369,6 @@ public final class io/sentry/Sentry {
 	public static fun pushScope ()Lio/sentry/ISentryLifecycleToken;
 	public static fun removeExtra (Ljava/lang/String;)V
 	public static fun removeTag (Ljava/lang/String;)V
-	public static fun reportFullDisplayed ()V
 	public static fun reportFullyDisplayed ()V
 	public static fun setCurrentHub (Lio/sentry/IHub;)Lio/sentry/ISentryLifecycleToken;
 	public static fun setCurrentScopes (Lio/sentry/IScopes;)Lio/sentry/ISentryLifecycleToken;

diff --git a/sentry/src/main/java/io/sentry/IScopes.java b/sentry/src/main/java/io/sentry/IScopes.java
@@ -657,14 +657,6 @@ void setSpanContext(
    */
   void reportFullyDisplayed();
 
-  /**
-   * @deprecated See {@link IScopes#reportFullyDisplayed()}.
-   */
-  @Deprecated
-  default void reportFullDisplayed() {
-    reportFullyDisplayed();
-  }
-
   /**
    * Continue a trace based on HTTP header values. If no "sentry-trace" header is provided a random
    * trace ID and span ID is created.

diff --git a/sentry/src/main/java/io/sentry/Sentry.java b/sentry/src/main/java/io/sentry/Sentry.java
@@ -1085,15 +1085,6 @@ public static void reportFullyDisplayed() {
     getCurrentScopes().reportFullyDisplayed();
   }
 
-  /**
-   * @deprecated See {@link Sentry#reportFullyDisplayed()}.
-   */
-  @Deprecated
-  @SuppressWarnings("InlineMeSuggester")
-  public static void reportFullDisplayed() {
-    reportFullyDisplayed();
-  }
-
   /** the metrics API for the current Scopes */
   @NotNull
   @ApiStatus.Experimental

diff --git a/sentry/src/test/java/io/sentry/ScopesTest.kt b/sentry/src/test/java/io/sentry/ScopesTest.kt
@@ -1975,13 +1975,6 @@ class ScopesTest {
         assertTrue(called)
     }
 
-    @Test
-    fun `reportFullDisplayed calls reportFullyDisplayed`() {
-        val scopes = spy(generateScopes())
-        scopes.reportFullDisplayed()
-        verify(scopes).reportFullyDisplayed()
-    }
-
     @Test
     fun `continueTrace creates propagation context from headers and returns transaction context if performance enabled`() {
         val scopes = generateScopes()

diff --git a/sentry/src/test/java/io/sentry/SentryTest.kt b/sentry/src/test/java/io/sentry/SentryTest.kt
@@ -714,17 +714,6 @@ class SentryTest {
         verify(scopes).reportFullyDisplayed()
     }
 
-    @Test
-    fun `reportFullDisplayed calls reportFullyDisplayed`() {
-        val scopes = mock<IScopes>()
-        Sentry.init {
-            it.dsn = dsn
-        }
-        Sentry.setCurrentScopes(scopes)
-        Sentry.reportFullDisplayed()
-        verify(scopes).reportFullyDisplayed()
-    }
-
     @Test
     fun `ignores executorService if it is closed`() {
         var sentryOptions: SentryOptions? = null