[JSC] parseMemberExpression() should create NewExpr with expression s…

…tart location

https://bugs.webkit.org/show_bug.cgi?id=268022
<rdar://problem/121544630>

Reviewed by Yusuke Suzuki and Justin Michaud.

This was the idea of https://bugs.webkit.org/show_bug.cgi?id=267728, yet unfortunately in-between
implementation revisions the case of correct location for `new new new C` did regress.

* Source/JavaScriptCore/parser/Parser.cpp:
(JSC::Parser<LexerType>::parseMemberExpression):

Canonical link: https://commits.webkit.org/273440@main

Source/JavaScriptCore/parser/Parser.cpp

@@ -5311,14 +5311,13 @@ template <class TreeBuilder> TreeExpression Parser<LexerType>::parseMemberExpres
 {
     TreeExpression base = 0;
     JSTextPosition expressionStart = tokenStartPosition();
-    JSTokenLocation startLocation = tokenLocation();
+    JSTokenLocation location = tokenLocation();
     Vector<JSTextPosition> newTokenStartPositions;
     while (match(NEW)) {
         newTokenStartPositions.append(tokenStartPosition());
         next();
     }
     size_t newCount = newTokenStartPositions.size();
-    JSTokenLocation location = tokenLocation();
 
     bool baseIsSuper = match(SUPER);
     bool previousBaseWasSuper = false;
@@ -5337,7 +5336,7 @@ template <class TreeBuilder> TreeExpression Parser<LexerType>::parseMemberExpres
                 semanticFailIfFalse(!closestOrdinaryFunctionScope->isGlobalCode() || isFunctionEvalContextType || isClassFieldInitializer, "new.target is not valid inside arrow functions in global code");
                 currentScope()->setInnerArrowFunctionUsesNewTarget();
             }
-            base = context.createNewTargetExpr(startLocation);
+            base = context.createNewTargetExpr(location);
             newCount--;
             next();
         } else {