favicons not displayed after import of bookmarks from Safari / Chrome

[LaurenWags]

Description

I imported a small amount of bookmarks (7 bookmarks and 2 folders) from Safari. Favicons for the imported bookmarks were not displayed after being imported.

Steps to Reproduce

1. Have bookmarks set up in safari and verify you can see their favicons. Close safari.
2. Import bookmarks from Safari.

Actual result:
Favicons were not displayed after import:

Expected result:
Favicons should be displayed after import of bookmarks.

Reproduces how often: [What percentage of the time does it reproduce?] 100%

Brave Version

about:brave info:
Brave | 0.19.23
OS Release | 16.7.0
Muon | 4.4.25

Reproducible on current live release:
Yes, can reproduce on 0.18.36, back to 0.16.6

Additional Information

cc: @kjozwiak

[darkdh]

importer.on('add-favicons' in app/importer.js is not being hit which means muon doesn't emit this event

[kjozwiak]

@NejcZdovc is this the same issue that I was experiencing in #10325 (review)? I believe it was fixed in that PR as well but I could be wrong.

@LaurenWags can you see if this is happening in the 0.20.x branch?

[LaurenWags]

@kjozwiak still experiencing this with 0.20.6

[bsclifton]

+1 from @comodoro via #12996

After install and importing bookmarks from Chrome everything imported correctly, but favicons have been the generic grey sheets without image. This is on the bookmarks toolbar as well as the bookmarks menu.

Current Brave 0.20.30, Windows 10.

[eljuno]

+1 from community. Imported from Chrome https://community.brave.com/t/bookmarks-bar-showing-as-a-list/24130?u=eljuno

[rebron]

Closing and marking wontfix. Reopen if still an issue in brave-core.