Fix an ErrorProne warning. ArrayList almost certainly performs better.
@@ -33,11 +33,11 @@ import com.google.javascript.rhino.JSTypeExpression;
import com.google.javascript.rhino.Node;
import com.google.javascript.rhino.Token;
import com.google.javascript.rhino.jstype.TernaryValue;
+import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
import java.util.HashMap;
import java.util.HashSet;
-import java.util.LinkedList;
import java.util.List;
import java.util.Map;
import java.util.Set;
@@ -2836,7 +2836,7 @@ public final class NodeUtilTest extends TestCase {
Scope functionScope = scopeCreator.createScope(functionNode, globalScope);
Map<String, Var> allVariables = new HashMap<>();
- List<Var> orderedVars = new LinkedList<>();
+ List<Var> orderedVars = new ArrayList<>();
NodeUtil.getAllVarsDeclaredInFunction(
allVariables, orderedVars, compiler, scopeCreator, functionScope);
Set<String> keySet = new HashSet<>(Arrays.asList("a", "b", "c", "z", "x", "y"));
@@ -2863,7 +2863,7 @@ public final class NodeUtilTest extends TestCase {
Scope functionScope = scopeCreator.createScope(functionNode, globalScope);
Map<String, Var> allVariables = new HashMap<>();
- List<Var> orderedVars = new LinkedList<>();
+ List<Var> orderedVars = new ArrayList<>();
NodeUtil.getAllVarsDeclaredInFunction(
allVariables, orderedVars, compiler, scopeCreator, functionScope);
Set<String> keySet = new HashSet<>(Arrays.asList("x", "y", "z", "a", "b", "c"));
