Improve favicon handling to cache only HTTP response codes 200-499, and not timeouts

Sometimes these external sites plain suck and can't return a favicon in under 2 seconds (our current timeout). Consider not caching results that timeout or error. Alternatively consider increasing the timeout so that if a their server is slow we'll wait for it once but then give up and cache after that. Alternatively consider making a subsequent HTTP request if the first one times out.