feat(browser): Mark errors caught from TryCatch integration as unhandled (#8890)

Change the mechanisms of errors caught from the `TryCatch` integration to report the error as unhandled.

Detailed write-up in #8890

Author: Lms24

packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/test.ts

@@ -17,7 +17,7 @@ sentryTest(
       value: 'event_listener_error',
       mechanism: {
         type: 'instrument',
-        handled: true,
+        handled: false,
       },
       stacktrace: {
         frames: expect.any(Array),

packages/browser-integration-tests/suites/sessions/update-session/test.ts

@@ -17,7 +17,7 @@ sentryTest('should update session when an error is thrown.', async ({ getLocalTe
   expect(updatedSession).toBeDefined();
   expect(updatedSession.init).toBe(false);
   expect(updatedSession.errors).toBe(1);
-  expect(updatedSession.status).toBe('ok');
+  expect(updatedSession.status).toBe('crashed');
   expect(pageloadSession.sid).toBe(updatedSession.sid);
 });
 

packages/browser/src/integrations/trycatch.ts

@@ -113,7 +113,7 @@ function _wrapTimeFunction(original: () => void): () => number {
     args[0] = wrap(originalCallback, {
       mechanism: {
         data: { function: getFunctionName(original) },
-        handled: true,
+        handled: false,
         type: 'instrument',
       },
     });
@@ -134,7 +134,7 @@ function _wrapRAF(original: any): (callback: () => void) => any {
             function: 'requestAnimationFrame',
             handler: getFunctionName(original),
           },
-          handled: true,
+          handled: false,
           type: 'instrument',
         },
       }),
@@ -160,7 +160,7 @@ function _wrapXHR(originalSend: () => void): () => void {
                 function: prop,
                 handler: getFunctionName(original),
               },
-              handled: true,
+              handled: false,
               type: 'instrument',
             },
           };
@@ -220,7 +220,7 @@ function _wrapEventTarget(target: string): void {
                 handler: getFunctionName(fn),
                 target,
               },
-              handled: true,
+              handled: false,
               type: 'instrument',
             },
           });
@@ -239,7 +239,7 @@ function _wrapEventTarget(target: string): void {
               handler: getFunctionName(fn),
               target,
             },
-            handled: true,
+            handled: false,
             type: 'instrument',
           },
         }),

packages/browser/test/integration/suites/builtins.js

@@ -182,7 +182,7 @@ describe('wrapped built-ins', function () {
 
         assert.deepEqual(summary.events[0].exception.values[0].mechanism, {
           type: 'instrument',
-          handled: true,
+          handled: false,
           data: {
             function: 'onreadystatechange',
           },
@@ -237,7 +237,7 @@ describe('wrapped built-ins', function () {
 
         assert.deepEqual(summary.events[0].exception.values[0].mechanism, {
           type: 'instrument',
-          handled: true,
+          handled: false,
         });
       }
     });
@@ -277,7 +277,7 @@ describe('wrapped built-ins', function () {
         assert.oneOf(target, ['Node', 'EventTarget']);
         assert.deepEqual(summary.events[0].exception.values[0].mechanism, {
           type: 'instrument',
-          handled: true,
+          handled: false,
           data: {
             function: 'addEventListener',
           },
@@ -313,7 +313,7 @@ describe('wrapped built-ins', function () {
           assert.oneOf(target, ['Node', 'EventTarget']);
           assert.deepEqual(summary.events[0].exception.values[0].mechanism, {
             type: 'instrument',
-            handled: true,
+            handled: false,
             data: {
               function: 'addEventListener',
               handler: '<anonymous>',