Update: UsePlayerTasks - lifting skipping logic out of OcclusionLineOfSight to caller - left a couple more TODOs as I found a weirdness in the original code Only thing left in OcclusionLineOfSight is grid-cell caching and deduplication. Tests: none, discovered unit tests are broken since I upgraded occlusion cache version a month ago. Will fix and test later