brief-rss / brief

RSS reader extension for Firefox
Other
214 stars 44 forks source link

consider using blob URLs for favicons #495

Open tanriol opened 4 years ago

tanriol commented 4 years ago

data: URLs are expensive, and sometimes not in nice places. For example, it was found that a feedlist build and the corresponding favicon URL assignment can lock the parent Firefox process for multiple seconds as it receives the URLs and checks them against add-on webRequest and webNavigation handlers if one has add-ons that register handlers for the affected hooks.

This is partially related to #466, but can be much more severe.