[destructuring] Refactor duplicate parameter name detection.

src/parser.cc

Index: src/parser.cc
diff --git a/src/parser.cc b/src/parser.cc
index 28dcf1eb32e5240e2ecba7dfa4571e14287a91f0..7b62b383a8cb59fd1882ce44dbd109c62db4a72d 100644
--- a/src/parser.cc
+++ b/src/parser.cc
@@ -3793,11 +3793,9 @@ void ParserTraits::DeclareArrowFunctionParameters(
   parser_->scope_->RemoveUnresolved(expr->AsVariableProxy());
 
   bool is_rest = false;
-  bool is_duplicate = DeclareFormalParameter(scope, raw_name, is_rest);
-
-  if (is_duplicate && !duplicate_loc->IsValid()) {
-    *duplicate_loc = param_location;
-  }
+  ExpressionClassifier classifier;
+  DeclareFormalParameter(scope, raw_name, &classifier, is_rest);
+  *duplicate_loc = classifier.duplicate_formal_parameter_error().location;
 }