Revert "perf: use Uint8Array bitmaps for faster tag ID lookups"

[fb55]

Reverts #1679

Performance seems equivalent if not slightly worse, plus increases the code size.

[coveralls]

coverage: 95.746% (-0.009%) from 95.755%
when pulling 6813154 on revert-1679-bitmap-optimizations
into f7c310f on master.

[Copilot]

Pull request overview

This PR reverts the previously introduced Uint8Array bitmap optimization for tag ID membership checks in parse5, restoring Set-based lookups to keep performance neutral (or better) while reducing code size.

Changes:

• Replace bitmap-based scope checks in the open element stack with Set membership checks.
• Replace bitmap-based foreign-content “exit tags” lookup with a Set.
• Remove the bitmap helper module and the SVG benchmark script introduced with the optimization.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
packages/parse5/lib/parser/open-element-stack.ts	Reverts scope checking from bitmap lookups back to Set-based scoping element checks.
packages/parse5/lib/common/tag-id-bitmap.ts	Removes the bitmap implementation helper entirely.
packages/parse5/lib/common/foreign-content.ts	Reverts foreign-content exit-tag lookup from bitmap back to a Set.
bench/perf/svg-benchmark.js	Removes the SVG benchmark script that was added alongside the optimization.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Grammar in the comment: "Tags that causes exit from foreign content" should be "Tags that cause exit from foreign content".

Suggested change

	//Tags that causes exit from foreign content
	// Tags that cause exit from foreign content