Bug 1742628 - Part 3: Move AtomVector and FunctionBoxVector. r=tcampbell

And remove some outdated comments referring to the previous trace list
managed by the parser.

Depends on D131922

Differential Revision: https://phabricator.services.mozilla.com/D131923

js/src/frontend/NameAnalysisTypes.h

@@ -12,9 +12,6 @@
 
 #include <stdint.h>  // uint8_t, uint16_t, uint32_t
 
-#include "frontend/ParserAtom.h"     // TrivialTaggedParserAtomIndex
-#include "js/AllocPolicy.h"          // SystemAllocPolicy
-#include "js/Vector.h"               // Vector
 #include "vm/BindingKind.h"          // BindingKind, BindingLocation
 #include "vm/BytecodeFormatFlags.h"  // JOF_ENVCOORD
 #include "vm/BytecodeUtil.h"  // ENVCOORD_HOPS_BITS, ENVCOORD_SLOT_BITS, GET_ENVCOORD_HOPS, GET_ENVCOORD_SLOT, ENVCOORD_HOPS_LEN, JOF_OPTYPE, JSOp, LOCALNO_LIMIT
@@ -387,14 +384,6 @@ class NameLocation {
   }
 };
 
-// These types are declared here for BaseScript::CreateLazy.
-using AtomVector = Vector<TrivialTaggedParserAtomIndex, 24, SystemAllocPolicy>;
-
-class FunctionBox;
-// FunctionBoxes stored in this type are required to be rooted
-// by the parser
-using FunctionBoxVector = Vector<FunctionBox*, 24, SystemAllocPolicy>;
-
 }  // namespace frontend
 }  // namespace js
 

js/src/frontend/NameCollections.h

@@ -278,6 +278,10 @@ class VectorPool : public CollectionPool<RepresentativeVector,
   }
 };
 
+using AtomVector = Vector<TrivialTaggedParserAtomIndex, 24, SystemAllocPolicy>;
+
+using FunctionBoxVector = Vector<FunctionBox*, 24, SystemAllocPolicy>;
+
 class NameCollectionPool {
   InlineTablePool<AtomIndexMap> mapPool_;
   VectorPool<AtomVector> atomVectorPool_;