fix: validate PerformanceObserver observe options

Summary:
Harden `PerformanceObserver.observe()` option validation so invalid observer configurations fail deterministically before we register a native observer.

This fixes a few user-facing gaps in the current implementation:

- calling `observe()` with no options now throws the expected argument error instead of destructuring `undefined`
- the `entryTypes` / `type` validation messages now match the actual invalid condition
- empty `entryTypes: []` is rejected instead of silently creating an observer that observes nothing
- `entryTypes` combined with `buffered` is rejected before native registration, matching the existing `durationThreshold` guard

The goal is to prevent malformed performance observer setup from silently succeeding or failing with misleading errors in apps and performance tooling.

## Changelog:

[GENERAL] [FIXED] - Validate invalid PerformanceObserver observe option combinations before native registration

X-link: #57030

Reviewed By: javache

Differential Revision: D107361950

Pulled By: rubennorte

fbshipit-source-id: eb0614fc20541f3cc2a7acacbbef63394ffcd0a1

Author: ya-nsh

packages/react-native/src/private/webapis/performance/PerformanceObserver.js

@@ -201,18 +201,30 @@ export class PerformanceObserver {
     return observerHandle;
   }
 
-  #validateObserveOptions(options: PerformanceObserverInit): void {
-    const {type, entryTypes, durationThreshold} = options;
+  #validateObserveOptions(options: ?PerformanceObserverInit): void {
+    if (options == null) {
+      throw new TypeError(
+        "Failed to execute 'observe' on 'PerformanceObserver': 1 argument required, but only 0 present.",
+      );
+    }
+
+    const {type, entryTypes, durationThreshold, buffered} = options;
 
     if (!type && !entryTypes) {
       throw new TypeError(
-        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must not include both entryTypes and type arguments.",
+        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must include either entryTypes or type arguments.",
       );
     }
 
     if (entryTypes && type) {
       throw new TypeError(
-        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must include either entryTypes or type arguments.",
+        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must not include both entryTypes and type arguments.",
+      );
+    }
+
+    if (entryTypes && buffered != null) {
+      throw new TypeError(
+        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must not include both entryTypes and buffered arguments.",
       );
     }
 

packages/react-native/src/private/webapis/performance/__tests__/PerformanceObserver-itest.js

@@ -96,6 +96,31 @@ describe('PerformanceObserver', () => {
     expect(entries2.getEntries()[1]).toBe(measure);
   });
 
+  describe('observe()', () => {
+    it('rejects invalid entry type option combinations before registering with native', () => {
+      const callback = jest.fn();
+      const observer = new PerformanceObserver(callback);
+
+      // $FlowExpectedError[incompatible-call]
+      expect(() => observer.observe()).toThrow(
+        "Failed to execute 'observe' on 'PerformanceObserver': 1 argument required, but only 0 present.",
+      );
+      expect(() => observer.observe({})).toThrow(
+        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must include either entryTypes or type arguments.",
+      );
+      expect(() =>
+        observer.observe({entryTypes: ['mark'], type: 'measure'}),
+      ).toThrow(
+        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must not include both entryTypes and type arguments.",
+      );
+      expect(() =>
+        observer.observe({entryTypes: ['mark'], buffered: true}),
+      ).toThrow(
+        "Failed to execute 'observe' on 'PerformanceObserver': An observe() call must not include both entryTypes and buffered arguments.",
+      );
+    });
+  });
+
   describe('takeRecords()', () => {
     it('provides all buffered events and clears the buffer', () => {
       const callback = jest.fn();