Browser-laptop: Favicon of trakt.tv on new tab page

[darkdh]

Did you search for similar issues before submitting this one?
Yes
Describe the issue you encountered:
The favicon blends in with the red background
Expected behavior:
favicon of trakt.tv should be recognizable

• Platform (Win7, 8, 10? macOS? Linux distro?): all

• Brave Version: 0.12.10

• Screenshot if needed:

The right most red one is trakt.tv

Its favicon is

cc @bradleyrichter , @cezaraugusto , @bsclifton

[bsclifton]

Another great example of this one- youtube:

[bradleyrichter]

@cezaraugusto Is trying a 90% opacity on the BG color to keep the icon visible without changing the look too much.

[bsclifton]

Another similar issue:
https://github.com/brave/browser-laptop/issues/5858

In this case, the computed colors are different based on your login state (see pic). What's weird is that those colors may impact brand recognition

[bsclifton]

One way we could solve this would be to first look for the Microsoft attributes. On Twitter for example, they provide this:

<meta name="msapplication-TileImage" content="//abs.twimg.com/favicons/win8-tile-144.png"/>
<meta name="msapplication-TileColor" content="#00aced"/>

This is similar to the suggestion by Cezar in https://github.com/brave/browser-laptop/issues/5663

[cndouglas]

That's basically the suggestion in #5658.

[bsclifton]

@Liunkae yes! thanks for posting (I was searching and was unable to find that :smile: )

[luixxiul]

Test plan: https://github.com/brave/browser-laptop/pull/5984#issue-193167093