Avoid redunant cell <-> map conversions

The CellRegion and CellLayer classes prefer map-coords internally since they can be used more directly for array indexing, but their APIs all expose cell co-ordinates.

The lead to a particularly bad case where enumerating the cells of a region would do map->cell converions for you, only to use those cells as an indexer into a layer which would convert the cell back into a map co-ordinate. All the round trips are quite expensive since so many are performed.

This PR adds the ability to enumerate a region in map-coords and then uses this ability in several key places to index into the layer using those map-coords. I didn't fix every use since there's so many.
The LINQ would be ```footprint.Any(mapCoord => p.Shroud.IsVisibleTest(footprintRegion)(mapCoord.X, mapCoord.Y))```.

Since it's a hot path, I figured it would be best to avoid allocating here. It's annoying but since it's a closure you can't just cache the delegate or anything.