Handle strings when skipping function bodies (#8614)

Explicitly parse strings in the logic for mostly skipping function
bodies in the first parser phase. This avoids parentheses inside strings
causing the parser to skip too much or too little of the input.

Author: tlively

src/parser/context-decls.cpp

@@ -329,6 +329,11 @@ bool ParseDeclsCtx::skipFunctionBody() {
       }
       continue;
     }
+    // Avoid confusion due to parens inside strings by skipping strings as a
+    // unit.
+    if (in.takeString()) {
+      continue;
+    }
     in.take(1);
     in.advance();
   }

test/lit/wat-kitchen-sink.wast

@@ -5259,4 +5259,11 @@
    )
   )
  )
+
+ (func $paren-in-string
+   ;; We should not be tripped up by an extra close parenthesis inside a string.
+   (drop
+     (string.const ")")
+   )
+ )
 )