SurfaceManager: lazily construct surface maps on demand

[Zehvogel]

BEGINRELEASENOTES

• Changed SurfaceManager to only construct surfaces and their maps on demand.

ENDRELEASENOTES

@jmcarcell FYI :)

[github-actions]

Test Results

   14 files     14 suites   7h 8m 12s ⏱️
  365 tests   352 ✅ 0 💤 13 ❌
2 510 runs  2 473 ✅ 0 💤 37 ❌

For more details on these failures, see this check.

Results for commit 3f99f2a.

♻️ This comment has been updated with latest results.

[Zehvogel]

I tried to make the commits a bit more readable. I can remove the white space removing commit if you insist.

Overall the changes seem to work, but I don't know if they are useful for someone. The initial motivation was to speed-up loading of the CLD detector. However, as it turns out, there we also load in the next step a plugin that accesses all surfaces anyway so no performance gain can be observed 🤷

[andresailer]

At least this PR told me that something in dev3 is breaking DD4hep :/

[andresailer]

At least this PR told me that something in dev3 is breaking DD4hep :/

root-project/root#16167