connect - popup might not be needed

[mroz22]

Consider not opening the connect-popup if we can tell beforehand that we won't need it.

unskip skipped test in #14517

[Hannsek]

In which situation we currently open popup when we don't need it?

[martykan]

For example getFeatures for an already remembered device with granted permissions

[bsl]

Upvote. Coming from Ledger, I'm finding it disturbing that Metamask wants to open a Trezor domain and download over a meg of minified Javascript just to sign a tx. I don't understand what the rationale is, and I'm disturbed by how little discussion there is about what data if any is being sent when that page is opened. If that needs to happen once in order to grant permission to export public keys, that seems fine, but then please remember it and don't open this popup on every subsequent tx. Thanks.