browser: Fixed ignoreNextOnError

Author: kamilogorek

packages/browser/src/integrations/globalhandlers.ts

@@ -8,6 +8,7 @@ import {
   StackTrace as TraceKitStackTrace,
   subscribe,
 } from '../tracekit';
+import { shouldIgnoreOnError } from './helpers';
 
 /** Global handlers */
 export class GlobalHandlers implements Integration {
@@ -43,6 +44,9 @@ export class GlobalHandlers implements Integration {
       //   9: "    foo();"
       //   10: "  }"
       // ]
+      if (shouldIgnoreOnError()) {
+        return;
+      }
       captureEvent(this.eventFromGlobalHandler(stack));
     });
 

packages/browser/src/integrations/helpers.ts

@@ -1,16 +1,17 @@
 import { getCurrentHub } from '@sentry/hub';
-import { SentryEvent, SentryWrappedFunction } from '@sentry/types';
+import { Mechanism, SentryEvent, SentryWrappedFunction } from '@sentry/types';
 import { isFunction } from '@sentry/utils/is';
 import { htmlTreeAsString } from '@sentry/utils/misc';
 
 const debounceDuration: number = 1000;
 let keypressTimeout: number | undefined;
 let lastCapturedEvent: Event | undefined;
-let ignoreOnError: number = -1;
-
-// TODO: Fix `ignoreNextOnError`. Just temporary build fix for unused variable
-ignoreOnError = ignoreOnError + 1;
+let ignoreOnError: number = 0;
 
+/** JSDoc */
+export function shouldIgnoreOnError(): boolean {
+  return ignoreOnError > 0;
+}
 /** JSDoc */
 export function ignoreNextOnError(): void {
   // onerror should trigger before setTimeout