1889455 - Jump List favicons don't work for favicons that are 144x144

Status: NEW

(Core :: Widget: Win32, defect, P3)

Description

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

STR:

1. On Windows in Firefox, clear history, and open up the Browser Console
2. Put the following into the Browser Console (Ctrl-Shift-J to open) to repopulate the jump list:

const { WinTaskbarJumpList } = ChromeUtils.importESModule(
  "resource:///modules/WindowsJumpLists.sys.mjs"
);

WinTaskbarJumpList.update();

3. Check that the jump list does not have any history items in it, since history was removed in step 1.
4. Visit https://mozilla.org and https://www.youtube.com
5. Re-run the JS in the console from step 2. Do this twice.
6. Check the jump list and see that both the mozilla.org and youtube.com visits appear with their favicons.
7. Visit https://www.axios.com/2024/04/02/phone-while-driving-pay-fine?utm_source=pocket-newtab-en-us
8. Re-run the JS in the console from step 2 twice.

ER:

In the jump list, the Axios visit should have a favicon from that site.

AR:

The Axios visit has the default favicon instead of the Axios favicon.

This seems to be because the favicon from the Axios website is 140x140. We're doing the right thing and caching that favicon on the disk, and looking at the cache, the favicon is 140x140. I suspect that Windows is choking on that for some reason - maybe we're exceeding some internal limits that I don't see documented here or here. At any rate, it might be worthwhile to just hardcode some static size (48px seems to be what we do for smaller icons) and always scale to that. Or maybe better ask the system for the icon size it expects.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:rkraesig, could you have a look please?

For more information, please visit BugBot documentation.