Closed Bug 1490869 Opened 6 years ago Closed 6 years ago

favicon not updated until everything blown away and browser restarted

Categories

(Firefox :: Tabbed Browser, defect)

defect
Not set
normal

Tracking

()

RESOLVED INCOMPLETE
Tracking Status
firefox62 --- wontfix
firefox63 --- ?
firefox64 --- ?

People

(Reporter: dietrich, Unassigned)

References

Details

Filing report from a developer I met at a conference:

Firefox Quantum v 62.0 (64-bit) on Fedora Linux

Steps to reproduce:

1. Load a site in a tab in Firefox.
2. Update the favicon file on the server and restart apache.
3. Reload the tab.

  * Expected result: favicon on tab changes
  * Actual result: no change

4. Open new tab and navigate to the path of the favicon.

  * Expected result: see the new favicon in the tab and the document area
  * Actual result: old favicon in both places

5. Ctrl+F5 on the favicon page.

  * Expected result: see the new favicon in the tab and the document area
  * Actual result: new favicon in document area but tab remains the same

6. Return to the application page. Open console. Check that the request for the favicon is what I expect and has no funny headers. Clear the cache for the site, disable cache, and hard refresh.

  * Expected result: New favicon in tab
  * Actual result: Same old favicon

7. Navigate to the browser Preferences, Cookies and Site Data, and clear data. Exit prefs and reload page.

  * Expected result: New favicon in tab
  * Actual result: Same old favicon

8. Open a blank tab, close the application tab, reopen prefs and clear the browser cache again. Close Firefox and reopen it. Navigate to site.

Finally see new favicon. Sigh with relief.
(In reply to Dietrich Ayala (:dietrich) from comment #0)
> Filing report from a developer I met at a conference:
> 
> Firefox Quantum v 62.0 (64-bit) on Fedora Linux

How about 63? Favicon handling has been revamped in bug 1453751.
Depends on: 1453751
The fact that closing the app tab made a difference makes me wonder if the imglib cache is at play here...
Yeah this is highly likely to be the image loader cache being overly aggressive I think. It is likely fixed in 63 but would be nice to check and even have a test for it.
Hi, that developer reporting in. v63 hasn't landed in Fedora's dnf packages yet but if I have time I'll see if I can install it another way and give it a shot.

At Dietrich's request I attempted to reproduce the issue in Google Chrome 68.0.3440.106 {"arch":"x86-64","nacl_arch":"x86-64","os":"linux"} as well, and I ended up getting the new icon at step 5 rather than step 8.

I will also try reproducing it in IE or Edge in a little while if I am able to get my virtual machines to stop misbehaving. :)
My expectation is that you'll get the new favicon at step 3 in Firefox 63 assuming the server isn't telling us to cache the icon for a while.
David, any update?
Flags: needinfo?(davmillar)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INCOMPLETE
Flags: needinfo?(davmillar)
You need to log in before you can comment on or make changes to this bug.