User Time Zone feature results in DI loop

[mvarblow]

Describe the bug

After enabling the User Time Zone feature, the site hangs and no longer responds to new requests.

Orchard Core version

2.1, 2.1.1

To Reproduce

1. Clone the repo
2. Start the OrchardCore.Cms.Web project
3. Use the default recipe and complete setup
4. Login and go to the Features admin page
5. Enable the User Time Zone feature
6. The site hangs

Expected behavior

The site should continue to function, but with the user time zone feature enabled.

Logs and screenshots

After the site hangs, I can break in the debugger and inspect the call stack. It's truncated by VS to about 4000 call sites. This is repeated on the call stack over and over:

That repeated section of call stack includes this:

and this:

[mvarblow]

@MikeAlhayek, I think this PR introduced the problem.

#16942

Reverting that PR seems to resolve the problem.