Chromium Code Reviews
Description[Remote suggestions] Get favicon URLs from archive
Fetching favicons via content suggestions service is currently only
enabled for remote suggestions. The general API for getting the URL is
via content_suggestion.h which does not work for archived suggestions
(i.e. by Fetch More).
This CL introduces a quick fix to loop in remote suggestions provider
for the favicon URLs of suggestions not known to
ContentSuggestionsService. A cleaner fix requires more thinking /
refactoring.
This is a M59 merge candidate.
BUG=714025
Review-Url: https://codereview.chromium.org/2829963005
Cr-Commit-Position: refs/heads/master@{#466653}
(cherry picked from commit 8083bc5ed7a1989652884bec28bf775dd198b447)
Review-Url: https://codereview.chromium.org/2846503002 .
Cr-Commit-Position: refs/branch-heads/3071@{#219}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed: https://chromium.googlesource.com/chromium/src/+/0a1c2e959fc5b622ce4b82239c4afa5a56404d0b
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||