Bug 1547035 - Handle AssignmentNode->BinaryNode change from bug 1535471 in Reflect.parse. r=jorendorff

Differential Revision: https://phabricator.services.mozilla.com/D28890

--HG--
extra : moz-landing-system : lando

js/src/builtin/ReflectParse.cpp

@@ -2538,7 +2538,7 @@ bool ASTSerializer::classField(ClassField* classField, MutableHandleValue dst) {
                            .head()
                            ->as<UnaryNode>()
                            .kid()
-                           ->as<AssignmentNode>()
+                           ->as<BinaryNode>()
                            .right();
     // RawUndefinedExpr is the node we use for "there is no initializer". If one
     // writes, literally, `x = undefined;`, it will not be a RawUndefinedExpr

js/src/tests/jstests.list

@@ -10,10 +10,6 @@ skip script non262/String/normalize-generateddata-input.js # input data for othe
 slow script test262/built-ins/decodeURI/S15.1.3.1_A2.5_T1.js
 slow script test262/built-ins/decodeURIComponent/S15.1.3.2_A2.5_T1.js
 
-# Fields are not fully implemented yet
-# https://bugzilla.mozilla.org/show_bug.cgi?id=1499448
-skip script non262/reflect-parse/class-fields.js
-
 # Windows10-aarch64 fails certain tests.
 # https://bugzilla.mozilla.org/show_bug.cgi?id=1526003
 # https://bugzilla.mozilla.org/show_bug.cgi?id=1526012

js/src/tests/non262/reflect-parse/class-fields.js

@@ -1,4 +1,4 @@
-// |reftest| skip-if(!xulRuntime.shell)
+// |reftest| skip-if(!xulRuntime.shell||(function(){try{eval('c=class{x;}');return(false);}catch{return(true);}})())
 // Classes
 function testClassFields() {
     function constructor_(name) {