Revert NavigationSchool using modules back to legacy

Valgrind found that NavigationSchool obtained from the EventSetup
is not thread safe. Therefore we reverted all modules using that
instance back to legacy types until the issue can be solved.
Merge pull request #3515 from Dr15Jones/revertNavigationSchoolUsingModulesBackToLegacy

Multithreading -- Revert NavigationSchool using modules back to legacy