Vite should stop appending `?import` for data URLs

Describe the bug

Vite injects ?import query to data URL in dynamic import calls, which should be unexpected.

Reproduction

const blob = new Blob([/* */]， { type: '...' })
const url = URL.createObjectURL(blob)

;(async () => {
  await import(/* @vite-ignore */ url) // <== Vite injects `?import` query here.
})()

I hope if we add @vite-ignore magic comment, Vite won’t inject ?import query, otherwise it will break the URL and cause failing to load such resources.

The code of Vite below will mark “needs inject query”: https://github.com/vitejs/vite/blob/main/packages/vite/src/node/plugins/importAnalysis.ts#L412

Then, Vite will inject at runtime: https://github.com/vitejs/vite/blob/main/packages/vite/src/node/plugins/importAnalysis.ts#L413

I hope we can add a detection:

if (!hasViteIgnore) {
  needQueryInjectHelper = true
  str().overwrite(start, end, `__vite__injectQuery(${url}, 'import')`)
}

Edit: My another solution is to skip adding queries for URLs which start with blob: and data: at both compile-time and runtime.

System Info

Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:

  System:
    OS: Linux 5.11 Arch Linux
    CPU: (8) x64 AMD Ryzen 5 2500U with Radeon Vega Mobile Gfx
    Memory: 1.85 GB / 6.75 GB
    Container: Yes
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 15.12.0 - /usr/bin/node
    Yarn: 1.22.10 - /usr/bin/yarn
    npm: 7.6.3 - /usr/bin/npm
  Browsers:
    Firefox: 86.0.1
  npmPackages:
    @vitejs/plugin-vue: ^1.1.5 => 1.1.5 
    vite: ^2.1.0 => 2.1.2

Used package manager: npm

Logs

---

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

• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn’t already an issue that reports the same bug to avoid creating a duplicate.
• Provide a description in this issue that describes the bug.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it’s a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

[g-plane]

Minimal reproduction repository: https://github.com/rosaric/vite-bug-repro .

[g-plane]

Can I attempt to open a PR that adds a processing for URLs start with data: or blob:?