Don't erroneously revert to using /favicon.ico as the tab favicon on some pages when navigating back in the middle of a document load

RESOLVED INVALID

Status

()

Firefox
Tabbed Browser
RESOLVED INVALID
5 years ago
4 years ago

People

(Reporter: fryn, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
Steps to reproduce:
1. Navigate to a page on Wikipedia, e.g. http://en.wikipedia.org/wiki/Wikipedia#External_links
   - The page contains a <link rel="shortcut icon"/>, and the tab displays the image at that path, as verified with DOM Inspector. 
2. Navigate to another page, e.g. the "Video of TED Talk by Jimmy Wales on the birth of Wikipedia" link, that may take a few seconds to load.
3. Immediately after the page navigation is committed (the location bar changes value to page B's url) but before the page finishes loading, navigate back (e.g. via the back button or alt/cmd+left).

Expected result:
The tab once again displays the image at the <link rel="shortcut icon"> path.

Actual result:
The tab displays the image at /favicon.ico.
(Verify with DOM Inspector.)

I'm not sure if this happens on any other websites.

If this is indeed a legitimate bug on our side, I suspect that this bug has something to do with mTabProgressListener's onStateChange, in particular the following lines:
  if (!this.mBrowser.mIconURL)
    this.mTabBrowser.useDefaultIcon(this.mTab);
(Reporter)

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.