Merge pull request #27 from amplitude/AMP-105355

AMP-105355

.changeset/no-neg-lookbehind.md

@@ -0,0 +1,6 @@
+---
+'@amplitude/rrweb-snapshot': patch
+'@amplitude/rrweb': patch
+---
+
+Replay: Replace negative lookbehind in regexes from css parser as it causes issues with Safari 16

.changeset/silent-plants-perform.md

@@ -0,0 +1,5 @@
+---
+'@amplitude/rrweb': patch
+---
+
+Return early for child same origin frames

packages/rrweb-snapshot/src/css.ts

@@ -424,6 +424,17 @@ export function parse(css: string, options: ParserOptions = {}): Stylesheet {
    * Parse selector.
    */
 
+  // originally from https://github.com/NxtChg/pieces/blob/3eb39c8287a97632e9347a24f333d52d916bc816/js/css_parser/css_parse.js#L46C1-L47C1
+  const selectorMatcher = new RegExp(
+    '^((' +
+      [
+        /[^\\]"(?:\\"|[^"])*"/.source, // consume any quoted parts (checking that the double quote isn't itself escaped)
+        /[^\\]'(?:\\'|[^'])*'/.source, // same but for single quotes
+        '[^{]',
+      ].join('|') +
+      ')+)',
+  );
+
   function selector() {
     whitespace();
     while (css[0] == '}') {
@@ -432,8 +443,7 @@ export function parse(css: string, options: ParserOptions = {}): Stylesheet {
       whitespace();
     }
 
-    // Use match logic from https://github.com/NxtChg/pieces/blob/3eb39c8287a97632e9347a24f333d52d916bc816/js/css_parser/css_parse.js#L46C1-L47C1
-    const m = match(/^(((?<!\\)"(?:\\"|[^"])*"|(?<!\\)'(?:\\'|[^'])*'|[^{])+)/);
+    const m = match(selectorMatcher);
     if (!m) {
       return;
     }
@@ -869,8 +879,8 @@ export function parse(css: string, options: ParserOptions = {}): Stylesheet {
         name +
         '\\s*((?:' +
         [
-          '(?<!\\\\)"(?:\\\\"|[^"])*"',
-          "(?<!\\\\)'(?:\\\\'|[^'])*'",
+          /[^\\]"(?:\\"|[^"])*"/.source, // consume any quoted parts (checking that the double quote isn't itself escaped)
+          /[^\\]'(?:\\'|[^'])*'/.source, // same but for single quotes
           '[^;]',
         ].join('|') +
         ')+);',