Fix: PerformanceAndroidEventProcessor uses up to date isTracingEnabled set on Configuration callback (#1786)

Author: marandaneto

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 * Feat: Add datasource tracing with P6Spy (#1784)
 * Fix: ActivityFramesTracker does not throw if Activity has not been added (#1782)
+* Fix: PerformanceAndroidEventProcessor uses up to date isTracingEnabled set on Configuration callback (#1786)
 
 ## 5.2.4
 

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

@@ -19,16 +19,15 @@
 /** Event Processor responsible for adding Android metrics to transactions */
 final class PerformanceAndroidEventProcessor implements EventProcessor {
 
-  private final boolean tracingEnabled;
-
   private boolean sentStartMeasurement = false;
 
   private final @NotNull ActivityFramesTracker activityFramesTracker;
+  private final @NotNull SentryAndroidOptions options;
 
   PerformanceAndroidEventProcessor(
       final @NotNull SentryAndroidOptions options,
       final @NotNull ActivityFramesTracker activityFramesTracker) {
-    tracingEnabled = options.isTracingEnabled();
+    this.options = Objects.requireNonNull(options, "SentryAndroidOptions is required");
     this.activityFramesTracker =
         Objects.requireNonNull(activityFramesTracker, "ActivityFramesTracker is required");
   }
@@ -37,7 +36,7 @@ final class PerformanceAndroidEventProcessor implements EventProcessor {
   public synchronized @NotNull SentryTransaction process(
       @NotNull SentryTransaction transaction, @Nullable Object hint) {
 
-    if (!tracingEnabled) {
+    if (!options.isTracingEnabled()) {
       return transaction;
     }