Skip to content

Commit 9cb8201

Browse files
committed
Move from ApiStatus.Internal to ApiStatus.Experimental
1 parent cb9e7f6 commit 9cb8201

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

sentry-android-replay/src/main/java/io/sentry/android/replay/screenshot/CanvasStrategy.kt

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import io.sentry.SentryOptions
3232
import io.sentry.android.replay.ScreenshotRecorderCallback
3333
import io.sentry.android.replay.ScreenshotRecorderConfig
3434
import io.sentry.android.replay.util.submitSafely
35+
import io.sentry.util.IntegrationUtils
3536
import java.util.LinkedList
3637
import java.util.WeakHashMap
3738
import java.util.concurrent.ScheduledExecutorService
@@ -69,6 +70,8 @@ internal class CanvasStrategy(
6970
init {
7071
processingThread.start()
7172
processingHandler = Handler(processingThread.looper)
73+
74+
IntegrationUtils.addIntegrationToSdkVersion("ReplayCanvasStrategy")
7275
}
7376

7477
private val pictureRenderTask = Runnable {
@@ -105,7 +108,9 @@ internal class CanvasStrategy(
105108
canvas.drawColor(Color.BLACK, PorterDuff.Mode.CLEAR)
106109
holder.picture.draw(canvas)
107110
surface.unlockCanvasAndPost(canvas)
108-
} finally {}
111+
} catch (t: Throwable) {
112+
options.logger.log(SentryLevel.ERROR, "Canvas Strategy: picture render failed", t)
113+
}
109114
}
110115

111116
@SuppressLint("UnclosedTrace")

sentry/src/main/java/io/sentry/ScreenshotStrategyType.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import org.jetbrains.annotations.ApiStatus;
44

55
/** Enum representing the available screenshot strategies for replay recording. */
6-
@ApiStatus.Internal
6+
@ApiStatus.Experimental
77
public enum ScreenshotStrategyType {
88
/** Uses Canvas-based rendering for capturing screenshots */
99
CANVAS,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@ public void setDebug(final boolean debug) {
355355
*
356356
* @return the screenshot strategy
357357
*/
358-
@ApiStatus.Internal
358+
@ApiStatus.Experimental
359359
public @NotNull ScreenshotStrategyType getScreenshotStrategy() {
360360
return screenshotStrategy;
361361
}
@@ -365,7 +365,7 @@ public void setDebug(final boolean debug) {
365365
*
366366
* @param screenshotStrategy the screenshot strategy to use
367367
*/
368-
@ApiStatus.Internal
368+
@ApiStatus.Experimental
369369
public void setScreenshotStrategy(final @NotNull ScreenshotStrategyType screenshotStrategy) {
370370
this.screenshotStrategy = screenshotStrategy;
371371
}

0 commit comments

Comments
 (0)