feat: use import.meta.url instead of self.location

[sapphi-red]

Description

Use import.meta.url instead of replacing them with self.location.

fixes #10842

Additional context

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the PR Title Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[sapphi-red]

/foo gets replaced with /@fs/foo by vite:asset-import-meta-url plugin. Maybe we should add /* @vite-ignore */ for new URL as well?

[bluwy]

Using /* @vite-ignore */ works for me 👍

[sapphi-red]

I'll make a separate PR for that.

[bluwy]

Isn't that changing this code to use /* @vite-ignore */ instead, or is there more to it?

[sapphi-red]

new URL(foo, import.meta.url) doesn't support /* @vite-ignore */ now. So adding support for that is needed.

[bluwy]

Ah okay I thought it's already supported. Good to merge then.

[bluwy]

LGTM. I assume this should also work in non-esmodule workers as we have

vite/packages/vite/src/node/plugins/worker.ts

Lines 188 to 200 in badaadb

	export function webWorkerPostPlugin(): Plugin {
	return {
	name: 'vite:worker-post',
	resolveImportMeta(property, { chunkId, format }) {
	// document is undefined in the worker, so we need to avoid it in iife
	if (property === 'url' && format === 'iife') {
	return 'self.location.href'
	}
	return null
	},
	}
	}

[sapphi-red]

Yep, it works by that part.