Option #1 icon tinting sgtm

(a related thought, we should probably think about solving this generically since when we have theme-color tabs in the switcher, we might run into this same thing non-incognito mode too)

Alex - can you share steps to get to what you're seeing? The icon is tinted properly (to match the close icon) for me on Stable 50.0.266.89 and Dev 52.0.273.0.

Hwi corrected the error of my ways! I was looking at the incognito tab itself. Check out the attached. I say we'd want to match wherever the faded page icon is used (chrome:// pages maybe?) to be the same as the incognito tab one.

Assigning to Ted to queue up for his team as time allows (unless this belongs to another eng group, not quite sure.)

Deleted: woahtwotints.png 117 KB

	woahtwotints.png 117 KB View Download

Some notes I have after looking into this:

- The asset that the NTP uses and the actual default icon are two different assets in chrome. (src/chrome/android/java/res/drawable... and src/ui/resources/...)

- It should be pretty easy to solve this problem for incognito, I would just need an incognito version of the asset on the native side (and probably an android only code block). That code is in GetDefaultFavicon(...) in chrome_content_browser_client.cc.

- Having this work for theme colors will likely require copying the logic from java and making a c++ version (the two icon sources are pretty far removed from eachother).

Removing myself and adding cleer@ as primary design contact

Not sure if we want to approach this issue right now so I'm marking this as available. It seems like we would need to override Android's default icon specifically (probably in java) and maybe even theme it since there won't be a catch-all color that has perfect contrast with all possible theme colors.