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.