Once T253432: Create a User factory is completed, UserManager can be removed from CheckUser
Background
The CheckUser extension introduced a UserManager service in order to avoid calling a static method on the User object (User::newFromName), and instead use the dependency injection pattern.
Since the UserIdentityLookup has now been implemented in core, we can inject this instead of UserManager, and remove the UserManager class.
Acceptance criteria
- Anywhere the UserManager was injected in the CheckUser extension, the UserIdentityLookup (from MediaWiki core) is injected instead
- Calls to UserManager->idFromName() are replaced with calls to UserIdentityLookup->getUserIdentityByName()->getId() (although note that getUserIdentityByName may return null, so this case should be handled)
- The UserFactory class is removed