DescriptionRemove preferences among favicon types when choosing large icons
Make all UIs to use the largest available icon regardless of its type.
Prior to this patch, many UI chose favicons over touch icons, as long as
some minimum size was achieved (UI-dependent, often 16x16 on Android due
to default_favicon_min_size).
Sample affected site: m.youtube.com on Android's bookmark or history UI:
- Before: UI chose 16x16 favicon: http://s.ytimg.com/yts/favicon-vflz7uhzw.ico
- After: UI chose 144x144 touch icon:
http://s.ytimg.com/yts/mobile/img/apple-touch-icon-144x144-precomposed-vflwq-hLZ.png
This change will (in a later CL) allow reducing the number of downloaded
favicons on mobile, because currently two large icons are downloaded,
the largest one per type (one for non-touch icons and the other one for
touch icons), since the FaviconService's API allow clients to define
arbitrary type preferences.
BUG=698671
Patch Set 1 #Patch Set 2 : Fix iOS. #Patch Set 3 : Leftovers. #Patch Set 4 : Rebased. #Messages
Total messages: 12 (12 generated)
|