Revert "fix(favicon): record an icon that failed before hydration"

The failure it recorded was never the cause: Links read its favicons twice because the server rows
were discarded before they hydrated, so no ref on them ever ran. Links, its one server-rendered
caller, holds the globe either way, and every other caller mounts after hydration.

This reverts commit 0a4901f.

Author: waleedlatif1

apps/sim/components/favicon.tsx

@@ -32,25 +32,21 @@ interface FaviconProps {
 export function Favicon({ host, fallback, className, onLoad }: FaviconProps) {
   const [, rerender] = useReducer((failures: number) => failures + 1, 0)
   if (!host || knownMisses.has(host)) return fallback
-  const recordMiss = () => {
-    knownMisses.set(host, true)
-    rerender()
-  }
   return (
     <img
       ref={(image) => {
-        if (!image?.complete) return
-        if (image.naturalWidth > 0) onLoad?.()
-        // A failure before hydration fires no onError, and a remount would ask for the icon again.
-        else void image.decode().catch(recordMiss)
+        if (onLoad && image?.complete && image.naturalWidth > 0) onLoad()
       }}
       src={faviconSrc(host)}
       alt=''
       loading='lazy'
       decoding='async'
       className={cn('rounded-sm', className)}
       onLoad={onLoad}
-      onError={recordMiss}
+      onError={() => {
+        knownMisses.set(host, true)
+        rerender()
+      }}
     />
   )
 }