@newrelic/browser-agent 1.237.1 → 1.239.0

package/dist/cjs/common/config/state/init.js

@@ -17,8 +17,6 @@ const model = () => {
     }
   };
   return {
-    allow_bfcache: true,
-    // *cli - temporary feature flag for BFCache work
     privacy: {
       cookies_enabled: true
     },
@@ -27,7 +25,8 @@ const model = () => {
       deny_list: undefined,
       block_internal: true,
       enabled: true,
-      harvestTimeSeconds: 10
+      harvestTimeSeconds: 10,
+      autoStart: true
     },
     distributed_tracing: {
       enabled: undefined,
@@ -46,32 +45,39 @@ const model = () => {
     obfuscate: undefined,
     jserrors: {
       enabled: true,
-      harvestTimeSeconds: 10
+      harvestTimeSeconds: 10,
+      autoStart: true
     },
     metrics: {
-      enabled: true
+      enabled: true,
+      autoStart: true
     },
     page_action: {
       enabled: true,
-      harvestTimeSeconds: 30
+      harvestTimeSeconds: 30,
+      autoStart: true
     },
     page_view_event: {
-      enabled: true
+      enabled: true,
+      autoStart: true
     },
     page_view_timing: {
       enabled: true,
       harvestTimeSeconds: 30,
-      long_task: false
+      long_task: false,
+      autoStart: true
     },
     session_trace: {
       enabled: true,
-      harvestTimeSeconds: 10
+      harvestTimeSeconds: 10,
+      autoStart: true
     },
     harvest: {
       tooManyRequestsDelay: 60
     },
     session_replay: {
       // feature settings
+      autoStart: true,
       enabled: false,
       harvestTimeSeconds: 60,
       sampleRate: 0.1,
@@ -110,23 +116,25 @@ const model = () => {
     },
     spa: {
       enabled: true,
-      harvestTimeSeconds: 10
+      harvestTimeSeconds: 10,
+      autoStart: true
     }
   };
 };
 const _cache = {};
+const missingAgentIdError = 'All configuration objects require an agent identifier!';
 function getConfiguration(id) {
-  if (!id) throw new Error('All configuration objects require an agent identifier!');
+  if (!id) throw new Error(missingAgentIdError);
   if (!_cache[id]) throw new Error("Configuration for ".concat(id, " was never set"));
   return _cache[id];
 }
 function setConfiguration(id, obj) {
-  if (!id) throw new Error('All configuration objects require an agent identifier!');
+  if (!id) throw new Error(missingAgentIdError);
   _cache[id] = (0, _configurable.getModeledObject)(obj, model());
   (0, _nreum.gosNREUMInitializedAgents)(id, _cache[id], 'config');
 }
 function getConfigurationValue(id, path) {
-  if (!id) throw new Error('All configuration objects require an agent identifier!');
+  if (!id) throw new Error(missingAgentIdError);
   var val = getConfiguration(id);
   if (val) {
     var parts = path.split('.');

package/dist/cjs/common/config/state/runtime.js

@@ -11,10 +11,6 @@ var _runtime = require("../../constants/runtime");
 var _env = require("../../constants/env.npm");
 const model = {
   buildEnv: _env.BUILD_ENV,
-  bytesSent: {},
-  // Used for SM to capture body bytes sent per endpoint
-  queryBytesSent: {},
-  // Used for SM to capture query parameter bytes sent per endpoint
   customTransaction: undefined,
   disabled: false,
   distMethod: _env.DIST_METHOD,

package/dist/cjs/common/constants/env.cdn.js

@@ -12,7 +12,7 @@ exports.VERSION = exports.DIST_METHOD = exports.BUILD_ENV = void 0;
 /**
  * Exposes the version of the agent
  */
-const VERSION = "1.237.1";
+const VERSION = "1.239.0";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/constants/env.npm.js

@@ -12,7 +12,7 @@ exports.VERSION = exports.DIST_METHOD = exports.BUILD_ENV = void 0;
 /**
  * Exposes the version of the agent
  */
-const VERSION = "1.237.1";
+const VERSION = "1.239.0";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/constants/runtime.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.supportsSendBeacon = exports.isiOS = exports.isWorkerScope = exports.isIE = exports.isBrowserScope = exports.initialLocation = exports.iOS_below16 = exports.globalScope = exports.ffVersion = void 0;
+exports.supportsSendBeacon = exports.isiOS = exports.isWorkerScope = exports.isIE = exports.isBrowserScope = exports.initialLocation = exports.iOSBelow16 = exports.globalScope = exports.ffVersion = void 0;
 /**
  * @file Contains constants about the environment the agent is running
  * within. These values are derived at the time the agent is first loaded.
@@ -36,8 +36,8 @@ const isiOS = /iPad|iPhone|iPod/.test(navigator.userAgent);
  * This was fixed in v16+.
  */
 exports.isiOS = isiOS;
-const iOS_below16 = isiOS && typeof SharedWorker === 'undefined';
-exports.iOS_below16 = iOS_below16;
+const iOSBelow16 = isiOS && typeof SharedWorker === 'undefined';
+exports.iOSBelow16 = iOSBelow16;
 const ffVersion = (() => {
   const match = navigator.userAgent.match(/Firefox[/\s](\d+\.\d+)/);
   if (Array.isArray(match) && match.length >= 2) {

package/dist/cjs/common/constants/shared-channel.js

@@ -11,7 +11,9 @@ exports.sharedChannel = void 0;
  */
 
 let onReplayReady;
-const sessionReplayInitialized = new Promise(resolve => onReplayReady = resolve);
+const sessionReplayInitialized = new Promise(resolve => {
+  onReplayReady = resolve;
+});
 const sharedChannel = Object.freeze({
   onReplayReady,
   sessionReplayInitialized

package/dist/cjs/common/deny-list/deny-list.js

@@ -64,7 +64,7 @@ function setDenyList(denyListConfig) {
       host = url;
       pathname = '';
     }
-    let [hostname, port] = host.split(':');
+    let [hostname] = host.split(':');
     denyList.push({
       hostname,
       pathname
@@ -81,10 +81,7 @@ function domainMatchesPattern(pattern, domain) {
   if (pattern.length > domain.length) {
     return false;
   }
-  if (domain.indexOf(pattern) === domain.length - pattern.length) {
-    return true;
-  }
-  return false;
+  return domain.indexOf(pattern) === domain.length - pattern.length;
 }
 
 /**
@@ -105,8 +102,5 @@ function comparePath(pattern, path) {
   if (pattern === '') {
     return true;
   }
-  if (pattern === path) {
-    return true;
-  }
-  return false;
+  return pattern === path;
 }

package/dist/cjs/common/drain/drain.js

@@ -55,7 +55,6 @@ function drain() {
   let agentIdentifier = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : '';
   let featureName = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : 'feature';
   curateRegistry(agentIdentifier);
-
   // If the feature for the specified agent is not in the registry, that means the instrument file was bypassed.
   // This could happen in tests, or loaders that directly import the aggregator. In these cases it is safe to
   // drain the feature group immediately rather than waiting to drain all at once.
@@ -74,6 +73,7 @@ function drain() {
     items.sort((a, b) => a[1].priority - b[1].priority);
     items.forEach(_ref2 => {
       let [group] = _ref2;
+      registry[agentIdentifier].delete(group);
       drainGroup(group);
     });
   }

package/dist/cjs/common/event-emitter/contextual-ee.js

@@ -3,30 +3,29 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.ee = void 0;
-exports.getOrSetContext = getOrSetContext;
+exports.ee = exports.contextId = void 0;
 var _nreum = require("../window/nreum");
 var _getOrSet = require("../util/get-or-set");
-var _mapOwn = require("../util/map-own");
 var _config = require("../config/config");
+var _eventContext = require("./event-context");
+var _bundleId = require("../ids/bundle-id");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
  */
 
-var ctxId = 'nr@context';
-
+// create a unique id to store event context data for the current agent bundle
+const contextId = "nr@context:".concat(_bundleId.bundleId);
 // create global emitter instance that can be shared among bundles
-let nr = (0, _nreum.gosNREUM)();
-var globalInstance;
+exports.contextId = contextId;
+const globalInstance = ee(undefined, 'globalEE');
+
+// Only override the exposed event emitter if one has not already been exposed
 exports.ee = globalInstance;
-if (nr.ee) {
-  exports.ee = globalInstance = nr.ee;
-} else {
-  exports.ee = globalInstance = ee(undefined, 'globalEE');
+const nr = (0, _nreum.gosNREUM)();
+if (!nr.ee) {
   nr.ee = globalInstance;
 }
-function EventContext() {}
 function ee(old, debugId) {
   var handlers = {};
   var bufferGroupMap = {};
@@ -45,27 +44,27 @@ function ee(old, debugId) {
   }
   var emitter = {
     on: addEventListener,
-    addEventListener: addEventListener,
-    removeEventListener: removeEventListener,
-    emit: emit,
+    addEventListener,
+    removeEventListener,
+    emit,
     get: getOrCreate,
-    listeners: listeners,
-    context: context,
+    listeners,
+    context,
     buffer: bufferEventsByGroup,
     abort,
     aborted: false,
-    isBuffering: isBuffering,
+    isBuffering,
     debugId,
     backlog: isolatedBacklog ? {} : old && typeof old.backlog === 'object' ? old.backlog : {}
   };
   return emitter;
   function context(contextOrStore) {
-    if (contextOrStore && contextOrStore instanceof EventContext) {
+    if (contextOrStore && contextOrStore instanceof _eventContext.EventContext) {
       return contextOrStore;
     } else if (contextOrStore) {
-      return (0, _getOrSet.getOrSet)(contextOrStore, ctxId, getNewContext);
+      return (0, _getOrSet.getOrSet)(contextOrStore, contextId, () => new _eventContext.EventContext(contextId));
     } else {
-      return getNewContext();
+      return new _eventContext.EventContext(contextId);
     }
   }
   function emit(type, args, contextOrStore, force, bubble) {
@@ -112,12 +111,13 @@ function ee(old, debugId) {
     return emitters[name] = emitters[name] || ee(emitter, name);
   }
   function bufferEventsByGroup(types, group) {
-    var eventBuffer = getBuffer();
+    const eventBuffer = getBuffer();
+    group = group || 'feature';
 
     // do not buffer events if agent has been aborted
     if (emitter.aborted) return;
-    (0, _mapOwn.mapOwn)(types, function (i, type) {
-      group = group || 'feature';
+    Object.entries(types || {}).forEach(_ref => {
+      let [_, type] = _ref;
       bufferGroupMap[type] = group;
       if (!(group in eventBuffer)) {
         eventBuffer[group] = [];
@@ -135,14 +135,6 @@ function ee(old, debugId) {
     return emitter.backlog;
   }
 }
-
-// get context object from store object, or create if does not exist
-function getOrSetContext(obj) {
-  return (0, _getOrSet.getOrSet)(obj, ctxId, getNewContext);
-}
-function getNewContext() {
-  return new EventContext();
-}
 function abort() {
   globalInstance.aborted = true;
   globalInstance.backlog = {};

package/dist/cjs/common/event-emitter/event-context.js

@@ -0,0 +1,12 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.EventContext = void 0;
+class EventContext {
+  constructor(contextId) {
+    this.contextId = contextId;
+  }
+}
+exports.EventContext = EventContext;

package/dist/cjs/common/harvest/harvest-scheduler.js

@@ -8,7 +8,6 @@ var submitData = _interopRequireWildcard(require("../util/submit-data"));
 var _sharedContext = require("../context/shared-context");
 var _harvest = require("./harvest");
 var _eol = require("../unload/eol");
-var _config = require("../config/config");
 var _sessionEntity = require("../session/session-entity");
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
@@ -43,7 +42,7 @@ class HarvestScheduler extends _sharedContext.SharedContext {
     this.harvest = new _harvest.Harvest(this.sharedContext);
 
     // unload if EOL mechanism fires
-    (0, _eol.subscribeToEOL)(this.unload.bind(this), (0, _config.getConfigurationValue)(this.sharedContext.agentIdentifier, 'allow_bfcache')); // TO DO: remove feature flag after rls stable
+    (0, _eol.subscribeToEOL)(this.unload.bind(this));
 
     /* Flush all buffered data if session resets and give up retries. This should be synchronous to ensure that the correct `session` value is sent.
       Since session-reset generates a new session ID and the ID is grabbed at send-time, any delays or retries would cause the payload to be sent under
@@ -110,7 +109,7 @@ class HarvestScheduler extends _sharedContext.SharedContext {
       if (!submitMethod) return false;
       const retry = !opts?.unload && submitMethod === submitData.xhr;
       payload = this.opts.getPayload({
-        retry: retry
+        retry
       });
       if (!payload) {
         if (this.started) {
@@ -146,7 +145,6 @@ class HarvestScheduler extends _sharedContext.SharedContext {
     if (this.started) {
       this.scheduleHarvest();
     }
-    return;
   }
   onHarvestFinished(opts, result) {
     if (this.opts.onFinished) {

package/dist/cjs/common/harvest/harvest.js

@@ -56,7 +56,8 @@ class Harvest extends _sharedContext.SharedContext {
       isFinalHarvest: spec.opts?.unload
     });
     const options = {
-      retry: !spec.opts?.unload && submitMethod === submitData.xhr
+      retry: !spec.opts?.unload && submitMethod === submitData.xhr,
+      isFinalHarvest: spec.opts?.unload === true
     };
     const payload = this.createPayload(spec.endpoint, options);
     const caller = this.obfuscator.shouldObfuscate() ? this.obfuscateAndSend.bind(this) : this._send.bind(this);
@@ -130,7 +131,8 @@ class Harvest extends _sharedContext.SharedContext {
       }
       return false;
     }
-    let url = "".concat(this.getScheme(), "://").concat(info.errorBeacon).concat(endpoint !== 'rum' ? "/".concat(endpoint) : '', "/1/").concat(info.licenseKey);
+    const endpointURLPart = endpoint !== 'rum' ? "/".concat(endpoint) : '';
+    let url = "".concat(this.getScheme(), "://").concat(info.errorBeacon).concat(endpointURLPart, "/1/").concat(info.licenseKey);
     if (customUrl) url = customUrl;
     if (raw) url = "".concat(this.getScheme(), "://").concat(info.errorBeacon, "/").concat(endpoint);
     const baseParams = !raw && includeBaseParams ? this.baseQueryString() : '';
@@ -158,11 +160,6 @@ class Harvest extends _sharedContext.SharedContext {
       // If body is null, undefined, or an empty object or array, send an empty string instead
       body = '';
     }
-
-    // Get bytes harvested per endpoint as a supportability metric. See metrics aggregator (on unload).
-    agentRuntime.bytesSent[endpoint] = (agentRuntime.bytesSent[endpoint] || 0) + body?.length || 0;
-    // Get query bytes harvested per endpoint as a supportability metric. See metrics aggregator (on unload).
-    agentRuntime.queryBytesSent[endpoint] = (agentRuntime.queryBytesSent[endpoint] || 0) + fullUrl.split('?').slice(-1)[0]?.length || 0;
     const headers = [];
     headers.push({
       key: 'content-type',

package/dist/cjs/common/ids/bundle-id.js

@@ -0,0 +1,19 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.bundleId = void 0;
+var _uniqueId = require("./unique-id");
+/**
+ * @file Contains a unique identifier for the running agent bundle
+ * when loaded.
+ * @copyright 2023 New Relic Corporation. All rights reserved.
+ * @license Apache-2.0
+ */
+
+/**
+ * Provides a unique id for the current agent bundle
+ */
+const bundleId = (0, _uniqueId.generateUuid)();
+exports.bundleId = bundleId;

package/dist/cjs/common/serialize/bel-serializer.js

@@ -34,7 +34,7 @@ function getAddStringContext(agentIdentifier) {
   function addString(str) {
     if (typeof str === 'undefined' || str === '') return '';
     var obfuscator = new _obfuscate.Obfuscator({
-      agentIdentifier: agentIdentifier
+      agentIdentifier
     });
     str = String(str);
     if (obfuscator.shouldObfuscate()) str = obfuscator.obfuscateString(str);

package/dist/cjs/common/session/session-entity.js

@@ -48,33 +48,37 @@ class SessionEntity {
    * expiresMs and inactiveMs are used to "expire" the session, but can be overridden in the constructor. Pass 0 to disable expiration timers.
    */
   constructor(opts) {
-    this.setup(opts);
-  }
-  setup(_ref) {
-    let {
+    const {
       agentIdentifier,
       key,
-      storage,
-      value = (0, _uniqueId.generateRandomHexString)(16),
-      expiresMs = _constants.DEFAULT_EXPIRES_MS,
-      inactiveMs = _constants.DEFAULT_INACTIVE_MS
-    } = _ref;
+      storage
+    } = opts;
     if (!agentIdentifier || !key || !storage) {
       throw new Error("Missing required field(s):".concat(!agentIdentifier ? ' agentID' : '').concat(!key ? ' key' : '').concat(!storage ? ' storage' : ''));
     }
     this.agentIdentifier = agentIdentifier;
     this.storage = storage;
     this.state = {};
-    this.sync(model);
 
     // key is intended to act as the k=v pair
     this.key = key;
+    this.ee = _contextualEe.ee.get(agentIdentifier);
+    (0, _wrap.wrapEvents)(this.ee);
+    this.setup(opts);
+  }
+  setup(_ref) {
+    let {
+      value = (0, _uniqueId.generateRandomHexString)(16),
+      expiresMs = _constants.DEFAULT_EXPIRES_MS,
+      inactiveMs = _constants.DEFAULT_INACTIVE_MS
+    } = _ref;
+    this.state = {};
+    this.sync(model);
+
     // value is intended to act as the primary value of the k=v pair
     this.state.value = value;
     this.expiresMs = expiresMs;
     this.inactiveMs = inactiveMs;
-    this.ee = _contextualEe.ee.get(agentIdentifier);
-    (0, _wrap.wrapEvents)(this.ee);
 
     // the first time the session entity class is instantiated, we check the storage API for an existing
     // object. If it exists, the values inside the object are used to inform the timers that run locally.

package/dist/cjs/common/storage/first-party-cookies.js

@@ -13,22 +13,23 @@ class FirstPartyCookies {
       var match = document.cookie.match(new RegExp('(^| )' + name + '=([^;]+)'));
       if (match) return match[2];
     } catch (err) {
+      // Error is ignored
       return '';
     }
   }
   set(key, value) {
     try {
-      const cookie = "".concat(key, "=").concat(value, "; Domain=").concat(domain, "; Path=/");
+      const cookie = "".concat(key, "=").concat(value, "; Domain=").concat(this.domain, "; Path=/");
       document.cookie = cookie;
     } catch (err) {
-      return;
+      // Error is ignored
     }
   }
   remove(key) {
     try {
-      return document.cookie = "".concat(key, "=; expires=Thu, 01 Jan 1970 00:00:00 UTC; Domain=").concat(domain, "; Path=/");
+      document.cookie = "".concat(key, "=; expires=Thu, 01 Jan 1970 00:00:00 UTC; Domain=").concat(this.domain, "; Path=/");
     } catch (err) {
-      return;
+      // Error is ignored
     }
   }
 }

package/dist/cjs/common/storage/local-storage.js

@@ -11,6 +11,7 @@ class LocalStorage {
       // Cast it back to undefined if it doesnt exist
       return localStorage.getItem(key) || undefined;
     } catch (err) {
+      // Error is ignored
       return '';
     }
   }
@@ -19,14 +20,14 @@ class LocalStorage {
       if (value === undefined || value === null) return this.remove(key);
       return localStorage.setItem(key, value);
     } catch (err) {
-      return;
+      // Error is ignored
     }
   }
   remove(key) {
     try {
       localStorage.removeItem(key);
     } catch (err) {
-      return;
+      // Error is ignored
     }
   }
 }

package/dist/cjs/common/timer/interaction-timer.js

@@ -27,14 +27,16 @@ class InteractionTimer extends _timer.Timer {
     }
     if (_runtime.isBrowserScope && opts.ee) {
       if (opts.ee) {
+        this.ee = opts.ee;
         const debouncedRefresh = (0, _invoke.debounce)(this.refresh.bind(this), 500, {
           leading: true
         });
-        opts.ee.on('fn-end', evts => {
+        this.refreshHandler = evts => {
           if (opts.refreshEvents.includes(evts?.[0]?.type)) {
             debouncedRefresh();
           }
-        });
+        };
+        opts.ee.on('fn-end', this.refreshHandler);
       }
 
       // watch for the vis state changing.  If the page is hidden, the local inactivity timer should be paused
@@ -48,22 +50,28 @@ class InteractionTimer extends _timer.Timer {
       (0, _pageVisibility.subscribeToVisibilityChange)(state => {
         if (state === 'hidden') this.pause();
         // vis change --> visible is treated like a new interaction with the page
-        else {
-          this.refresh();
-          this.onResume(); // emit resume event after state updated
-        }
+        else this.resume();
       }, false, false, this.abortController?.signal);
     }
   }
   abort() {
     this.clear();
     this.abortController?.abort();
+    if (this.refreshHandler) {
+      this.ee.removeEventListener('fn-end', this.refreshHandler);
+      this.refreshHandler = this.ee = null;
+    }
   }
   pause() {
     this.onPause();
     clearTimeout(this.timer);
     this.remainingMs = this.initialMs - (Date.now() - this.startTimestamp);
   }
+  resume() {
+    this.refresh();
+    this.onResume(); // emit resume event after state updated
+  }
+
   refresh(cb, ms) {
     this.clear();
     this.timer = this.create(cb, ms);

package/dist/cjs/common/timing/nav-timing.js

@@ -35,7 +35,7 @@ var DOM_CONTENT_LOAD_EVENT = 'domContentLoadedEvent';
 var navTimingValues = [];
 exports.navTimingValues = navTimingValues;
 function getPntType(type) {
-  if (typeof type == 'number') return type;
+  if (typeof type === 'number') return type;
   const types = {
     navigate: undefined,
     reload: 1,

package/dist/cjs/common/unload/eol.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.subscribeToEOL = subscribeToEOL;
 var _eventListenerOpts = require("../event-listener/event-listener-opts");
-var _invoke = require("../util/invoke");
 var _runtime = require("../constants/runtime");
 var _pageVisibility = require("../window/page-visibility");
 /*
@@ -30,37 +29,12 @@ if (_runtime.isWorkerScope) {
  * Subscribes a provided callback to the time/event when the agent should treat it as end-of-life.
  * This is used, for example, to submit a final harvest and send all remaining data on best-effort.
  * @param {function} cb - func to run before or during the last reliable event or time of an env's life span
- * @param {boolean} allowBFCache - (temp) feature flag to gate new v1222 BFC support
  */
-function subscribeToEOL(cb, allowBFCache) {
+function subscribeToEOL(cb) {
   if (_runtime.isBrowserScope) {
-    if (allowBFCache) {
-      (0, _pageVisibility.subscribeToVisibilityChange)(cb, true); // when user switches tab or hides window, esp. mobile scenario
-      (0, _eventListenerOpts.windowAddEventListener)('pagehide', cb); // when user navigates away, and because safari iOS v14.4- doesn't fully support vis change
-      // --this ought to be removed once support for version below 14.5 phases out
-    } else {
-      var oneCall = (0, _invoke.single)(cb);
-
-      // Firefox has a bug wherein a slow-loading resource loaded from the 'pagehide'
-      // or 'unload' event will delay the 'load' event firing on the next page load.
-      // In Firefox versions that support sendBeacon, this doesn't matter, because
-      // we'll use it instead of an image load for our final harvest.
-      //
-      // Some Safari versions never fire the 'unload' event for pages that are being
-      // put into the WebKit page cache, so we *need* to use the pagehide event for
-      // the final submission from Safari.
-      //
-      // Generally speaking, we will try to submit our final harvest from either
-      // pagehide or unload, whichever comes first, but in Firefox, we need to avoid
-      // attempting to submit from pagehide to ensure that we don't slow down loading
-      // of the next page.
-      if (!_runtime.ffVersion || navigator.sendBeacon) {
-        (0, _eventListenerOpts.windowAddEventListener)('pagehide', oneCall);
-      } else {
-        (0, _eventListenerOpts.windowAddEventListener)('beforeunload', oneCall);
-      }
-      (0, _eventListenerOpts.windowAddEventListener)('unload', oneCall);
-    }
+    (0, _pageVisibility.subscribeToVisibilityChange)(cb, true); // when user switches tab or hides window, esp. mobile scenario
+    (0, _eventListenerOpts.windowAddEventListener)('pagehide', cb); // when user navigates away, and because safari iOS v14.4- doesn't fully support vis change
+    // --this ought to be removed once support for version below 14.5 phases out
   } else if (_runtime.isWorkerScope) {
     _runtime.globalScope.cleanupTasks.push(cb); // close() should run these tasks before quitting thread
   }