New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 24 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Jun 2010
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug
R-7
M-6

Blocked on:
issue 9881

Blocking:
issue chromium-os:3776
issue 185594

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment
link

Issue 9982: Favicons are not updated when the DOM changes the favicon

Reported by arv@chromium.org, Apr 10 2009 Project Member

Issue description

What steps will reproduce the problem?
1. Change the href attribute of a <link rel=icon>

What is the expected output? 

The favicon in the tab should be updated.

What do you see instead?

The favicon stays the same

http://www.favicon.cc/ provides a drawing surface and it updates the 
favicon by setting the data url of the link element. In Firefox the favicon 
updates in real time. Nothing happens to the favicon in Chromium.
 

Comment 1 by arv@chromium.org, Apr 10 2009

Blockedon: 9881

Comment 2 by yoah%bar...@gtempaccount.com, Jun 29 2009

If the link is removed from the dom and another one is added, the icon is changed. But 
if done async via setTimeout, the change sometimes works and sometimes it doesn't. See 
page http://www.bardavid.com/test/changefavicon.html. Works well in firefox.

Comment 3 by calebegg@gmail.com, Oct 1 2009

 Issue 18613  has been merged into this issue.

Comment 4 by bennyboy...@gmail.com, Dec 8 2009

I can't get the icon to change any way I try.

Comment 5 by oritm@chromium.org, Dec 18 2009

Labels: -Area-BrowserUI Area-UI-Features
Area-UI-Features label replaces Area-BrowserUI label

Comment 6 by arv@chromium.org, Feb 1 2010

Labels: -Pri-3 -Size-Medium -Area-UI-Features Pri-2 Area-WebKit
Status: Assigned

Comment 7 by karen@chromium.org, Feb 5 2010

Labels: Mstone-5

Comment 8 by hheimbue...@gmail.com, Feb 24 2010

Yoah, would you mind putting your demo back up? (Getting a 404 now.) I'd be really
interested how you did this. http://ajaxify.com/run/favicon/animate/ takes a similar
approach (not async, though), but it doesn't seem to work in Chrome 4.0.249.89.

Comment 9 by davemoore@chromium.org, Feb 24 2010

It wasn't my demo, and seems to be back up now.

Comment 10 by yoah@google.com, Feb 24 2010

I added back the files, which were accidentally deleted. The async case seems to be 
working consistently well now.

Comment 11 by davemoore@chromium.org, Feb 24 2010

I have a webkit patch I'm trying to get through that will allow us to solve this in 
chrome.

Comment 12 by davemoore@chromium.org, Apr 1 2010

Labels: -Mstone-5 Mstone-6

Comment 13 by aa@chromium.org, Apr 13 2010

 Issue 40545  has been merged into this issue.

Comment 14 by aa@chromium.org, Apr 13 2010

Labels: Feature-Extensions apps-related

Comment 15 by davemoore@chromium.org, Apr 21 2010

Labels: Iteration-2 R-7
Status: Started
The webkit patch went in (https://bugs.webkit.org/show_bug.cgi?id=33812). The chromium 
fix should be straight forward.

Comment 16 by erikkay@chromium.org, Apr 27 2010

I think you landed a fix for this, right?

Comment 17 by bennyboy...@gmail.com, May 19 2010

This seems to be fixed in 6.0.401.1 dev

Comment 18 by dhw@chromium.org, May 21 2010

 Issue 42195  has been merged into this issue.

Comment 19 by dhw@chromium.org, May 21 2010

Confirm fixed in Chrome 6.0.408.1 (Official Build 47574) dev

davemoore, can the status be updated?

Comment 20 by aa@chromium.org, Jun 15 2010

Labels: Pri-3

Comment 21 by karen@chromium.org, Jun 28 2010

Status: Fixed

Comment 22 by danor...@gmail.com, Oct 7 2011

 Issue #99549  demonstrates this same problem in certain situations.

Comment 23 Deleted

Comment 24 by rom...@gmail.com, Apr 30 2012

implementation seems buggy, see  Issue #121333 

Comment 25 by bugdroid1@chromium.org, Oct 13 2012

Project Member
Blockedon: -chromium:9881
Blocking: -chromium-os:3776 chromium-os:3776
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 26 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Area-WebKit -Mstone-6 -Feature-Extensions Cr-Content Cr-Platform-Extensions M-6

Comment 27 by bugdroid1@chromium.org, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Comment 28 by bugdroid1@chromium.org, Apr 6 2013

Project Member
Labels: -Cr-Content Cr-Blink

Sign in to add a comment