fix: remove favicon handling

[Barbapapazes]

🔗 Linked issue

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

I remove no more needed code since this (nuxt/framework#5274 (comment)) is resolved.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[netlify]

✅ Deploy Preview for nuxt-content canceled.

Built without sensitive environment variables

Name	Link
🔨 Latest commit	ac36ed9
🔍 Latest deploy log	https://app.netlify.com/sites/nuxt-content/deploys/64ad05a5e95da300082f1ad0

[Atinux]

Have you checked that when hitting /favicon.ico we don't have the document driven making a fetch call?

[Barbapapazes]

In dev or in prod, I did not notice any fetch to /favicon.ico. In order to view it, I added a favicon in the document-driven playground.

[Atinux]

The issue is actually when we don't have any public/favicon.ico, the browser will fetch it automatically.

[farnabaz]

I can confirm that nitro return empty gif image in case of missing favicon

https://github.com/unjs/nitro/blob/1b9f51b7955d2490b1fd8da73af40780f30a1235/src/runtime/renderer.ts#L18-L26