<div dir="ltr"><div>Not in ESR60; but wanted to let you know since there have been favicon issues in the past.</div><div><br></div><div><br></div><div><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Dave Townsend</b> <span dir="ltr"><<a href="mailto:dtownsend@mozilla.com">dtownsend@mozilla.com</a>></span><br>Date: Mon, Jun 25, 2018 at 6:02 PM<br>Subject: Favicons are now loaded in the content process<br>To: Firefox Dev <<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a>><br><br><br><div dir="ltr">I just landed a fairly major change to how favicons are loaded on mozilla-inbound and it looks like it might stick. Previously we relied on the places service and XUL image elements to load favicons from the main process but now we load the favicon data in the content process and then just re-use that data everywhere we need it, currently as a data URI.<div><br></div><div>This should mean we now honour CSP rules for favicons, expose favicon requests properly to webextensions and a few other things. But the change is significant enough that there might be some edge cases that changed.</div><div><br></div><div>If you notice any oddities around favicons in tabs, the new tab page or the all tabs menu please CC me on the bugs you file.</div></div>
<br>______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
<br></div><br></div></div>