Rewrite framed map tracker ticking

[Warriorrrr]

Rewrites the tracking code for framed maps to remove the use of the tickCarriedBy method and the HoldingPlayer class. the tickCarriedBy method contained a lot of code that did not apply to framed maps at all, and by moving the parts that did elsewhere, we can essentially skip it.

Closes #9597

[MrPowerGamerBR]

I've been running this patch on my server for ~1 week+ (after making my own makeshift patch that did only the ItemFrame getItem() optimization, only to find out that someone else already had made the same optimization but better) and I couldn't find any issues, and it also improves performance a bit if the server has a lot of item frames (which is my case for a Survival server) by avoiding calling getItem(). Before the patch, all getItem()calls were using 0.25ms every tick. (not a lot but for big servers every optimization is a big W)

The only thing is that this patch needs to be rebased (again) for the latest version for it to compile, because MapDecoration's getType() is now type(): SparklyPower/SparklyPaper@27b4f21

[MrPowerGamerBR]

Rebased for 1.20.6 https://github.com/SparklyPower/SparklyPaper/blob/ver/1.20.6/patches/server/0004-Rewrite-framed-map-tracker-ticking.patch