well, I think that if we have no renderedIcon in the descriptor, we really shuold try to fetch it, right ? The condition makes sense to me...

In the past, this would (probably ?) never happen but now this will happen if we have a default icon.

I think your condition doesn't work in all situations (for example: we never tried to render yet, so isDefaultIcon is false by default)