Don't use parens with optional chaining meber expressions (#2921)

src/fast-path.js

@@ -221,14 +221,6 @@ FastPath.prototype.needsParens = function(options) {
       return false;
     }
 
-    case "MemberExpression": {
-      return (
-        parent.type === "MemberExpression" &&
-        parent.object === node &&
-        node.optional
-      );
-    }
-
     case "SpreadElement":
     case "SpreadProperty":
       return (

tests/optional_chaining/__snapshots__/jsfmt.spec.js.snap

@@ -13,6 +13,7 @@ a?.[++x];
 a?.b.c(++x).d;
 a?.b[3].c?.(x).d;
 (a?.b).c;
+a?.b?.c;
 delete a?.b;
 
 a?.b[3].c?.(x).d.e?.f[3].g?.(y).h;
@@ -28,7 +29,8 @@ a?.();
 a?.[++x];
 a?.b.c(++x).d;
 a?.b[3].c?.(x).d;
-(a?.b).c;
+a?.b.c;
+a?.b?.c;
 delete a?.b;
 
 a?.b[3].c?.(x).d.e?.f[3].g?.(y).h;

tests/optional_chaining/chaining.js

@@ -10,6 +10,7 @@ a?.[++x];
 a?.b.c(++x).d;
 a?.b[3].c?.(x).d;
 (a?.b).c;
+a?.b?.c;
 delete a?.b;
 
 a?.b[3].c?.(x).d.e?.f[3].g?.(y).h;