Remove delete from SiDetectorManager and PixelDetectorManager using std::unique_ptr

Remove delete from SiDetectorManager and PixelDetectorManager using std::unique_ptr. Ownerships become clearer.

RunTier0Tests.py was fine. RunTier0Tests.log

This merge request affects 4 packages:

• InnerDetector/InDetDetDescr/InDetReadoutGeometry
• InnerDetector/InDetDetDescr/PixelGeoModel
• InnerDetector/InDetDetDescr/PixelReadoutGeometry
• InnerDetector/InDetDetDescr/SCT_GeoModel

Adding @goetz ,@stsuno ,@oda ,@battagl ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

changed the description

CI Result SUCCESS (hash 64c1d81c)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15514]

The changes are fine and there are no CI issues. Approving.

Jason (L1)

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

mentioned in commit 1c941cee

merged

added sweep:ignore label