Wallet HD path is wrong for legacy

[scmart]

When connecting to a Ledger S hardware wallet, and using "Legacy (MEW/MyCrypto)", only the first address is correct.

Steps to Reproduce

1. Connect Ledger Nano S
2. Add Account
3. Select Legacy (MEW/MyCrypto)

Actual result:

Addresses 2-5 (and probably beyond) are the wrong addresses.

Expected result:

Addresses 2-5 should match what MEW, MyCrypto, and metamask with the legacy option list.

Reproduces how often:

Always

Brave version (brave://version info)

Version 1.32.113 Chromium: 96.0.4664.45 (Official Build) (64-bit)
Ubuntu 20.04

• Can you reproduce this issue with the current release? Yes
• Can you reproduce this issue with the beta channel? Unknown
• Can you reproduce this issue with the nightly channel? Unknown

Miscellaneous Information:

A minor note, the UI experience switching between Ledger Live and Legacy is bad. It reverts to the connect to HW wallet view, where I though I had to reclick connect, so I did, followed by an error, before it then snaps back to the address list.

[srirambv]

cc: @onyb @darkdh

[scmart]

@srirambv Seems like this should be an easy and important fix. Hard to use the Brave wallet when most of my addresses aren't discoverable.

[scmart]

Still broken in Version 1.35.101

@spylogsster @srirambv @onyb @darkdh

[spylogsster]

will take a look tomorrow, thanks @scmart for the reminder

[bbondy]

Shouldn't we permanently have a third option now to have the other derivation path in case someone used those addresses and wants to get access again via a new import?

[spylogsster]

Shouldn't we permanently have a third option now to have the other derivation path in case someone used those addresses and wants to get access again via a new import?

follow-up issue #21171

[kjozwiak]

This was originally uplifted/scheduled to go into a 1.35.x HF but never ended up being released. It will go out via 1.36.x as we won't be having another 1.35.x release at this point.

[srirambv]

Brave	1.36.106 Chromium: 99.0.4844.45 (Official Build) (64-bit)
Revision	edbc0b8343c7b10fddb0e1b4efb280b0f6e38cab-refs/branch-heads/4844@{#788}
OS	☑️ Linux	☑️ Windows 11 Version Dev (Build 22523.1000)	☑️ macOS Version 12.0.1 (Build 21C52)
Verified test plan from brave/brave-core#12314 Verified import accounts with Legacy derivation path from MM and Brave Wallet works Verified imported accounts on both legacy and live are both same on MM and Brave Wallet