This class looks like it's about computing icons for download files.  Would it be better to name it and scope it more specifically, something like `FileTypeIconResolver` or so?