Lots of things inside and around Service Manager are pretty unpolished. This is a general metabug for tracking cleanup work, since it's a bunch of little things. Examples:
- The logic in ServiceManager::Connect is not unlike spaghetti
- ServiceManager has some unnecessary API surface like StartService()
- BackgroundServiceManager no longer needs to exist
- Many Service Manager tests can be rewritten and greatly simplified
given our more current APIs and test helpers.
- Random thought: we could support constexpr Identity construction
Comment 1 by rockot@google.com
, Nov 13Status: Assigned (was: Untriaged)