Fixed COMPUTED_FIELD_DEF to be deleted without any additional check in the RemoveNonDeclarations pass.

In the RemoveNonDeclarations `shouldTraverse()` method, the cases for `COMPUTED_PROPERTIES` and `COMPUTED_FIELD_DEF` fell into the same check which caused unassigned `COMPUTED_FIELD_DEF` to throw a null pointer exception. We now separate `COMPUTED_FIELD_DEF` into it's own check that will instantly delete the `COMPUTED_FIELD_DEF`.

PiperOrigin-RevId: 551238929

Author: Closure Team

src/com/google/javascript/jscomp/ijs/ConvertToTypedInterface.java

@@ -201,11 +201,13 @@ public boolean shouldTraverse(NodeTraversal t, Node n, Node parent) {
               return false;
           }
         case COMPUTED_PROP:
-        case COMPUTED_FIELD_DEF:
           if (!NodeUtil.isLhsByDestructuring(n.getSecondChild())) {
             NodeUtil.deleteNode(n, t.getCompiler());
           }
           return false;
+        case COMPUTED_FIELD_DEF:
+          NodeUtil.deleteNode(n, t.getCompiler());
+          return false;
         case THROW:
         case RETURN:
         case BREAK:

test/com/google/javascript/jscomp/ijs/ConvertToTypedInterfaceTest.java

@@ -73,6 +73,20 @@ public void testEmptyClass() {
     test("class x {;}", "class x {}");
   }
 
+  @Test
+  public void testComputedFieldNoRHS() {
+    test(
+        lines(
+            "const s = Symbol();", //
+            "class Foo { ",
+            "  [s]",
+            "}"),
+        lines(
+            "const s = Symbol();", //
+            "class Foo {",
+            "}"));
+  }
+
   @Test
   public void testSuperClassFields() {
     test(