Issue 905350 link

Starred by 2 users

Issue metadata

Status:	Assigned
Owner:	chrisha@chromium.org
Components:	UI>Browser>Sessions
EstimatedDays:	----
NextAction:	----
OS:	Linux , Windows , Chrome , Mac
Pri:	3
Type:	Bug

Duplicate favicon loading code in session restore

Project Member Reported by chrisha@chromium.org, Nov 14

Issue description

Somebody just pointed out to me that we restore favicons in two separate places:

https://cs.chromium.org/chromium/src/chrome/browser/sessions/session_restore_delegate.cc?q=session+restore+favicon+file:cc&dr=CSs&l=72

https://cs.chromium.org/chromium/src/chrome/browser/sessions/tab_loader.cc?q=tab_loader.cc&sq=package:chromium&g=0&l=195

This should be cleaned up.

Comment 1 by benhenry@google.com, Jan 11

Status: Assigned (was: Untriaged)

This issue has an owner, a component and a priority, but is still listed as untriaged or unconfirmed. By definition, this bug is triaged. Changing status to "assigned". Please reach out to me if you disagree with how I've done this.

►

Issue 905350 link

Issue metadata

Duplicate favicon loading code in session restore

Issue description

Comment 1 by benhenry@google.com, Jan 11

Comment 1 Deleted

Sign in to add a comment