[parser] Fix parse errors in static computed class properties

verwaest · V8 LUCI CQ · commit 3b1adc17c16b · 2024-09-18T15:56:55.000Z
Otherwise we'll try to rewrite a FailureExpression Bug: 366323452 Change-Id: Ief76a242a4b625762a496d9c8c816bc7156a9b37 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5872647 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Auto-Submit: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/main@{#96163}
diff --git a/src/parsing/parser-base.h b/src/parsing/parser-base.h
@@ -2676,7 +2676,7 @@ ParserBase<Impl>::ParseClassPropertyDefinition(ClassInfo* class_info,
       DCHECK_IMPLIES(prop_info->is_computed_name, !prop_info->is_private);
 
       if (prop_info->is_computed_name) {
-        if (next_info_id != PeekNextInfoId() &&
+        if (!has_error() && next_info_id != PeekNextInfoId() &&
             !(prop_info->is_static ? class_info->has_static_elements()
                                    : class_info->has_instance_members())) {
           impl()->ReindexComputedMemberName(name_expression);
diff --git a/test/mjsunit/regress/regress-366323452.js b/test/mjsunit/regress/regress-366323452.js
@@ -0,0 +1,7 @@
+// Copyright 2024 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+assertThrows(`new class {
+    static [{aaa(){}];
+};`);
