Re #8359. Avoid a memory leak when updateFacilities() is called.

It starts by clearing the list of facilities (which contains owned pointers
to FacilityInfo objects), but didn't delete the held objects.