fix: remove special handling for Accept: text/html

[sapphi-red]

Description

This special handling for Accept: text/html was added by 44a30d5 for #2051.

But the original issue seems to be working without this code. Also only checking for Accept: text/html is problematic (#15025).

So I removed this code. I think there won't be any problem, because when you access the path twice, the same behavior happens without this change.

fixes #17340

[sapphi-red]

I added a change to fix the test for #14626.

[patak-dev]

It seems it still works without that code because of:

• #10880

Throwing ERR_LOAD_URL is the same as returning null when calling from the transform middleware: https://github.com/vitejs/vite/pull/10880/files#diff-6d94d6934079a4f09596acc9d3f3d38ea426c6f8e98cd766567335d42679ca7cR227

I would say this is fine, there is a comment already there

// Let other middleware handle if we can't load the url via transformRequest

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/vite@20376

commit: fb70e98

[sapphi-red]

ladle's failure is caused by rollup/rollup#6012. I'll take a look at sveltekit's one.

[sapphi-red]

/ecosystem-ci run

[vite-ecosystem-ci]

📝 Ran ecosystem CI on fb70e98: Open

suite	result	latest scheduled
vite-plugin-react	✅ success	❌ failure
histoire	❌ failure	❌ failure
vike	❌ failure	❌ failure
one	❌ failure	❌ failure
vite-plugin-rsc	✅ success	❌ failure

✅ astro, nuxt, analogjs, marko, qwik, rakkas, laravel, quasar, vitest, sveltekit, vite-plugin-cloudflare, vite-setup-catalogue, waku, react-router, vite-plugin-svelte, ladle, unocss, storybook, vitepress, vite-plugin-vue, vite-environment-examples, vite-plugin-pwa, vuepress