Surface.h rm thread unsafe version of setMaterialLayer, move the const_cast to the one remaining code that actually needs them

Surface.h rm thread unsafe version of setMaterialLayer, move the const_cast to the one remaining code that actually needs them

Mentioning @sroe

This merge request affects 2 packages:

• Tracking/TrkDetDescr/TrkDetDescrTools
• Tracking/TrkDetDescr/TrkSurfaces

Affected files list will not be printed in this case

Adding @amorley as watcher

added Tracking master review-pending-level-1 labels

CI Result SUCCESS (hash be894f78)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 37669]

Changes look fine and the CI is good. Approving. Clara (L1)

added review-approved label and removed review-pending-level-1 label

mentioned in commit 90545bd4

merged

added sweep:ignore label