New issue
Advanced search Search tips

Issue 885260 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 8
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Use FaviconHelper for the tab strip

Project Member Reported by megjab...@chromium.org, Sep 18

Issue description

We should also think about a favicon fetching logic on the java side with FaviconHelper, updating the favicons in the model onFaviconChanged at some point down the line. Owned by the Mediator perhaps?
 
Status: Fixed (was: Assigned)
Landed here:

https://chromium.googlesource.com/experimental/chromium/src/+/cd02635b4750996100bc14a9aca62c0381db3127

Get favicons on startup using the FaviconHelper

Change-Id: I0969ce8ec6c56cf38e529cb0b589a407fc989318
Reviewed-on: https://chromium-review.googlesource.com/c/experimental/chromium/src/+/1250181

Sign in to add a comment