MuonChamberLayout create tile id vector only locally to make move operator safe
This tries to solve a spurious crash observed when fitting SeedMuon tracks. So far I was not able to reliably reproduce the crash, but I suspect that the memory holding some tile ids is freed twice. To avoid this the corresponding data member is removed and only local/temporary vectors are used to make the move operation safe. Also removes unnecessary unsigned and int mixtures to avoid casts.
The crash report can be found here.
Edited by Andre Gunther