Fix logic errors with Maps in ItemFrames. Fixes BUKKIT-3621

When a map is in an item frame it is incorrectly sending the map
data for the map to every player in the world, instead of only those
who are in range of the item frame. It also will execute other sections
of code that are unrelated to Item Frames every 9 out of 10 ticks.

This causes the server to do more than it needs to, causing pretty
significant performance loss, and is a waste of bandwidth.

This change makes the code use the correct HashSet of tracked players
instead, so it will not send the data to players who do not need it.
Additionally, it will ensure that the ItemFrame control statement
always returns true, and only the map specific parts are restricted
to 1 in 10 ticks.