1213284 - Favicons should be imported from Edge (EdgeHTML)

Status: RESOLVED WORKSFORME

(Firefox :: Migration, enhancement, P3)

Description

[Vasilica Tamas, QA [:vtamas]]

Reproducible on: Firefox 44.0a1, Firefox 43.0a2, Firefox 42 Beta 5 under Windows 

Prerequisites: Edge is installed and has:
- browsing history
- saved favorites

STR
1. Run Firefox and go to "Bookmarks -> Show All Bookmarks -> Import and Backup -> Import Data from Another Browser"
2. Select Edge from the "Import Settings and Data" window and then press "Next"
3. Select “Favorites” and then press "Next"
4. Press "Finish" and check the favicons of imported bookmarks.

ER
The favicons are automatically displayed after import process. 

AR
Firefox does not show the favicons.
See screenshot: http://i.imgur.com/EMRA7Kn.jpg

Additional notes:
- This issue is reproducible on Firefox 44.0a1 (2015-10-09), Firefox 43.0a2 (2015-10-09) and Firefox 42 Beta 5 (20151008162217) on Windows 10 64-bit
- It is also reproducible on Firefox 42 beta 4

Comment 1

[:Gijs (he/him)]

Changing severity to S3 because this doesn't block migration.

We don't import favicons from any browser right now, and implementing it would take significant amounts of work. (cf bug 326701, bug 711995, bug 718220).

Comment 2

[Ania]

Updating Steps to reproduce here, as Firefox interface has changed since this was reported:

Prerequisites: Edge is installed and has:

• browsing history
• saved bookmarks

Steps to reproduce:

1. Launch Firefox and run the import wizard via "File-> Import Data from Another Browser".
2. Select Edge from the list of available browsers in the import wizard.
3. Select “Bookmarks” and wait for the import to complete.
4. Check the favicons of imported bookmarks in "Bookmarks -> Manage Bookmarks", or bookmarks toolbar.

Expected result
The favicons are automatically displayed after the import process.

Actual result
Firefox does not show the favicons until a visit to a bookmarked website.

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Hi portiawu! So we've assigned you this one.

Modern Edge is based off of Chromium, so we assume that it stores favicons the same way as Chromium does (see https://docs.google.com/document/d/1U81uIPEZlY5KQavIb5sM6UM9nh1q41w_-wOIwOb_A1Q/edit#) - but we don't actually know that for sure. Can you verify that this is the case?

Comment 4

[portiawuu]

(In reply to Mike Conley (:mconley) (:⚙️) from comment #3)

Hi portiawu! So we've assigned you this one.

Modern Edge is based off of Chromium, so we assume that it stores favicons the same way as Chromium does (see https://docs.google.com/document/d/1U81uIPEZlY5KQavIb5sM6UM9nh1q41w_-wOIwOb_A1Q/edit#) - but we don't actually know that for sure. Can you verify that this is the case?

I don't use Modern Edge at all. I'll set some favs and give it a try :)

Comment 5

[portiawuu]

My Notes

https://docs.google.com/document/d/1P1ykj1sRi-w4_V_eaz8iJEPX6vX_38e0YVVkBPUIFEM/edit?usp=sharing

Comment 6

[Mike Conley (:mconley) (:⚙️)]

Importing favicons for bookmarks now works for Chrome-based browsers via bug 711995. This bug is now for EdgeHTML Edge favicon import.

Comment 7

[Rares Doghi, Desktop QA]

Cannot reproduce this issue in our latest Release 123.0.1 or our latest Nightly build 125.0a1 (2024-03-11).