Passing in the native currency as the "address" seems wrong 🤔 I see that we were already doing this for non-ETH native currencies though, for some reason 😕

The effect of this, and perhaps this was the intent, is to have a jazzicon display when there is a custom symbol for the custom network. Like this:

If no symbol is provided, it looks like this.

I agree that we should not do this, and instead should just show the new default of Identicon: <div style={getStyles(diameter)}></div>;

Now, the only problem with changing this is if some users have become acustomed to seeing the jazzicon for their particular custom network... so I think it'd be best not to break that in this PR.

Yeesh. So we're treating this as an address without validating that it's even remotely similar to an address. That's unfortunate.

Reminds me of this bug: #8252.

Leaving this alone for this PR seems sensible to me. I'm not sure we should be concerned about users being accustomed to this icon though. It seems very broken and confusing to me. Particularly because we use jazzicons for people, and we don't want tokens mistaken for people. Maybe we can discuss it in a future standup.