We currently cache the images found in metadata in the SDWebImageManager shared cache when we create the metadata. This cached images is keyed by the URL relating to the image. When we fetch the highlights from the database, we want to use the cached metadata image if it is present and default to the favicon if it is not.
Plan: * Update Query in SQLLiteHistoryRecommendations:getHighlights() to join view_history_id_favicon with the metadata table to fetch metadata for all highlights * Update Site object to contain a PageMetadata object * Update SQLLiteHistory:basicHistoryColumnFactory to create PageMetadata object if metadata present and add to returned Site * Update Highlights UI Code to check for presence of PageMetadata:MediaURL value. If present, this should be used as highlights image rather than favicon, otherwise favicon should be used * Update TopSites provider name display code to fetch provider name from Site:PageMetadata rather than from a property directly in Site.
Created attachment 8844903 [details] [review] Pull request
Comment on attachment 8844903 [details] [review] Pull request I didn't look too much into the SQL bits. But the rest of it looks good
Attachment #8844903 - Flags: review?(fpatel) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Comment on attachment 8844903 [details] [review] Pull request Left a comment about potential DB table creation issues.
You need to log in before you can comment on or make changes to this bug.