perf(functions): Refactor dedup() using Property#toHash()

[donmccurdy]

Refactor of the dedup function, using hashProperty function (from #1757) to avoid O(n²) iteration over properties.

Related:

• Fixes dedup: animation samplers #1722
• Fixes dedup: Group material before check #1708

PR Dependency Tree

• PR feat(functions): Add hashProperty(...) function #1757
  • PR chore(functions): Add dedup() benchmark #1760
    • PR perf(functions): Refactor dedup() using Property#toHash() #1761 👈

This tree was auto-generated by Charcoal

[donmccurdy]

UPDATE: Resolved.

The dedup+hash approach is currently failing for one case with skins, see:

• Various improvements to 'optimize' quality #1073
• optimize() output is not visually consistent for several samples #953
• https://github.com/KhronosGroup/glTF-Sample-Models/tree/main/2.0/RecursiveSkeletons

Need a unit test on this "recursive skinning" setup if possible - it would have been easy to miss!