refactor: Avoid passing params where not needed

[maflcko]

It seems confusing to pass chain params, consensus params, or a time function around when it is not needed.

Fix this by:

• Inlining the passed time getter function. I don't see a use case why this should be mockable.
• Using chainman.GetConsensus() or chainman.GetParams(), where possible.

[vincenzopalazzo]

ACK fa1b76a

[fanquake]

Concept ACK

[DrahtBot]

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

• #25073 (test: Cleanup miner_tests by MarcoFalke)
• #25064 ([kernel 2b/n] Add ChainstateManager::m_adjusted_time_callback by dongcarl)
• #24474 (Additional thread safety annotations for CNode/Peer by ajtowns)
• #24232 (assumeutxo: add init and completion logic by jamesob)
• #23443 (p2p: Erlay support signaling by naumenkogs)
• #21527 (net_processing: lock clean up by ajtowns)
• #21006 (rpc: reduce LOCK(cs_min) scope in rest_block: ~5 times as many requests per second by martinus)
• #18933 (rpc: Add submit option to generateblock by MarcoFalke)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

[promag]

Code review ACK fa1b76a.

[dongcarl]

post-merge ACK. This synergizes well with some of the upcoming libbitcoinkernel changes. We might actually be able to remove all references to a global CChainParams 😲 (no promises tho)