refactor: move HD code from CWallet to LegacyScriptPubKeyMan

[knst]

Issue being fixed or feature implemented

During backport bitcoin#17261 significant part of HD chain has been forgotten in CWallet due to our own implementation.

This PR do not change behaviour of HD wallets, it's just refactoring.

What was done?

This PR refactor HD wallets implementation:

• key related stuff is moved from CWallet to LegacyScriptPubKeyMan (to follow-up Make ScriptPubKeyMan an actual interface and the wallet to have multiple bitcoin/bitcoin#17261)
• refactored duplicated code between hdChain and hdCryptedChain
• modules hdchain and bip39 related moved to wallet/ module

How Has This Been Tested?

Run unit/functional tests

Breaking Changes

N/A

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated relevant unit/integration/functional/e2e tests
• I have made corresponding changes to the documentation
• I have assigned this pull request to a milestone

[github-actions]

This pull request has conflicts, please rebase.

[UdjinM6]

Looks good! 👍 Will test it a bit

[UdjinM6]

👍

light ACK

[UdjinM6]

utACK

[github-actions]

This pull request has conflicts, please rebase.

[UdjinM6]

re-utACK

[knst]

@knst knst force-pushed the hd-1 branch from d0f309f to b5f4411

rebased to be on top of post-merged master to develop

[PastaPastaPasta]

utACK