[bookmarks] Don't prefer touch icons on iOS
This fixes bookmark sync, which can otherwise upload bookmarks without
favicons on iOS.
The change makes iOS consistent with the behavior on other platforms
(most notably Android) without influencing the UI, because it doesn't
read favicons from BookmarkModel.
Note also that this achieves symmetry with the sync download path,
where remote favicons are cached locally as regular (non-touch)
favicons. This means those favicons will be reused if the bookmark
changes locally, when the changes are sync-ed.
This patch also allows simplifying the bookmark layers because the
whole predicate is no longer used. This will be addressed in follow-up
Commit-Queue: Mikel Astiz <email@example.com>
Reviewed-by: Sylvain Defresne <firstname.lastname@example.org>
Reviewed-by: Mark Cogan <email@example.com>
1 file changed