Closed
Bug 1426916
Opened 7 years ago
Closed 7 years ago
No favicon in a new tab created using Duplicate Tab
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
RESOLVED
DUPLICATE
of bug 1423256
Tracking | Status | |
---|---|---|
firefox57 | --- | unaffected |
firefox58 | --- | unaffected |
firefox59 | --- | affected |
People
(Reporter: magicp.jp, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
video/mp4
|
Details |
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:59.0) Gecko/20100101 Firefox/59.0 ID:20171222220251
Steps to reproduce:
1. Launch Nightly
2. Open a new tab if there is no new tab (about:newtab)
3. Right click on a new tab and select "Duplicate Tab" in context menus
Actual results:
No favicon in a new tab created using Duplicate Tab.
Expected results:
New tab have favicon even if using Duplicate Tab.
Regression range:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=1dfa2eb559ad896115dcd9ee13f63da6ed1fce18&tochange=28f5404cc95e8e911bb21545b73638473b4aecd9
Blocks: 1418130
Has Regression Range: --- → yes
Has STR: --- → yes
status-firefox57:
--- → unaffected
status-firefox58:
--- → unaffected
Comment 1•7 years ago
|
||
johannh, looks like removing the favicon from the page is causing various issues like this one for duplicating tabs, and bug 1423256 for restoring tabs, and bug 1422962 for new windows. Do we want to be special casing all of these places with the chrome provided favicon instead of having content suggest one with <link rel=icon>?
Blocks: 1422074
Flags: needinfo?(jhofmann)
Comment 2•7 years ago
|
||
(In reply to Ed Lee :Mardak from comment #1)
> johannh, looks like removing the favicon from the page is causing various
> issues like this one for duplicating tabs, and bug 1423256 for restoring
> tabs, and bug 1422962 for new windows. Do we want to be special casing all
> of these places with the chrome provided favicon instead of having content
> suggest one with <link rel=icon>?
Bug 1422962 is not caused by removing the favicon. We want to special case common pages like about:home, about:newtab and about:privatebrowsing but to avoid these other edge cases we could consider adding the <link rel=icon> back to the pages, as long as we know that it isn't re-introducing flicker in the common case.
Flags: needinfo?(jhofmann)
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
I have verified this bug was resolved by bug 1423256.
Comment 5•7 years ago
|
||
Thanks!
Assignee | ||
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•