ref: Add fast-path to fetchImpl

Author: kamilogorek

packages/browser/src/transports/fetch.ts

@@ -1,6 +1,6 @@
 import { eventToSentryRequest, sessionToSentryRequest } from '@sentry/core';
 import { Event, Response, SentryRequest, Session, TransportOptions } from '@sentry/types';
-import { getGlobalObject, logger, supportsReferrerPolicy, SyncPromise } from '@sentry/utils';
+import { getGlobalObject, isNativeFetch, logger, supportsReferrerPolicy, SyncPromise } from '@sentry/utils';
 
 import { BaseTransport } from './base';
 
@@ -45,8 +45,13 @@ type FetchImpl = typeof fetch;
  * Safari:  resource blocked by content blocker
  */
 function getNativeFetchImplementation(): FetchImpl {
-  // Make sure that the fetch we use is always the native one.
+  // Fast path to avoid DOM I/O
   const global = getGlobalObject<Window>();
+  // eslint-disable-next-line @typescript-eslint/unbound-method
+  if (isNativeFetch(global.fetch)) {
+    return global.fetch.bind(global);
+  }
+
   const document = global.document;
   // eslint-disable-next-line deprecation/deprecation
   if (typeof document?.createElement === `function`) {

packages/utils/src/supports.ts

@@ -73,7 +73,7 @@ export function supportsFetch(): boolean {
  * isNativeFetch checks if the given function is a native implementation of fetch()
  */
 // eslint-disable-next-line @typescript-eslint/ban-types
-function isNativeFetch(func: Function): boolean {
+export function isNativeFetch(func: Function): boolean {
   return func && /^function fetch\(\)\s+\{\s+\[native code\]\s+\}$/.test(func.toString());
 }