setTimeout is only needed in setupOnce

Author: billyvg

packages/replay/src/integration.ts

@@ -177,7 +177,16 @@ Sentry.init({ replaysOnErrorSampleRate: ${errorSampleRate} })`,
     }
 
     this._setup();
-    this._initialize();
+
+    // Once upon a time, we tried to create a transaction in `setupOnce` and it would
+    // potentially create a transaction before some native SDK integrations have run
+    // and applied their own global event processor. An example is:
+    // https://github.com/getsentry/sentry-javascript/blob/b47ceafbdac7f8b99093ce6023726ad4687edc48/packages/browser/src/integrations/useragent.ts
+    //
+    // So we call `this._initialize()` in next event loop as a workaround to wait for other
+    // global event processors to finish. This is no longer needed, but keeping it
+    // here to avoid any future issues.
+    setTimeout(() =>  this._initialize());
   }
 
   /**
@@ -248,21 +257,11 @@ Sentry.init({ replaysOnErrorSampleRate: ${errorSampleRate} })`,
    * Initializes replay.
    */
   protected _initialize(): void {
-    // Once upon a time, we tried to create a transaction in `setupOnce` and it would
-    // potentially create a transaction before some native SDK integrations have run
-    // and applied their own global event processor. An example is:
-    // https://github.com/getsentry/sentry-javascript/blob/b47ceafbdac7f8b99093ce6023726ad4687edc48/packages/browser/src/integrations/useragent.ts
-    //
-    // So we call `replay.initialize()` in next event loop as a workaround to wait for other
-    // global event processors to finish. This is no longer needed, but keeping it
-    // here to avoid any future issues.
-    setTimeout(() => {
-      if (!this._replay) {
-        return;
-      }
+    if (!this._replay) {
+      return;
+    }
 
-      this._replay.initializeSampling();
-    });
+    this._replay.initializeSampling();
   }
 
   /** Setup the integration. */