Each new xul window does a stat call to look for non-existant window specific icons
Categories
(Core :: XUL, enhancement)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: florian, Assigned: florian)
References
Details
(Keywords: perf, Whiteboard: [fxperf:p1])
Attachments
(2 files)
I'm looking a startup main thread I/O, and during early startup we stat browser\chrome\icons\default\default.ico when doing the starting the layout in toolkit/content/gfxsanity/sanityparent.html and then a second time when doing it in omni.ja!/chrome/toolkit/res/hiddenWindow.html
Later we stat browser\chrome\icons\default\main-window.ico when opening the browser window.
None of these files exist. Firefox doesn't provide window-specific icon files, and uses the application icon for all windows.
Thunderbird uses that feature though, so I suggest that we reverse the behavior: instead of looking for an icon file on disk by default, we should only do it if an "icon" attribute has been specified.
|
Assignee | |
Comment 1•6 years ago
|
||
|
|
Assignee | |
Comment 2•6 years ago
|
||
Pushed by fqueze@mozilla.com:
https://hg.mozilla.org/comm-central/rev/0ec9e8d998fc
adjust Thunderbird code to use the 'icon' attribute to set the icon of XUL windows, r=mkmelin.
|
|
Assignee | |
Updated•6 years ago
|
|
||
Comment 5•6 years ago
|
||
| bugherder | ||
What is the solution for people who were using main-window.ico to set a custom taskbar icon? That ability was of particular value where one wished to run multiple instances of Firefox, each with its own profile, and be able to easily distinguish between them by giving each a distinct taskbar icon. Since Firefox doesn't provide a way to do it via profiles alone, it was achieved with multiple installs, each with their own main-window.ico (inelegant, but at least it worked).
Description
•