search

nicolo-ribaudo / modules-import-hooks-refactor

https://nicolo-ribaudo.github.io/modules-import-hooks-refactor/
MIT License
0 stars 0 forks source link

Always all HostLoadImportedModule for import() #6

Closed nicolo-ribaudo closed 2 years ago

nicolo-ribaudo commented 2 years ago

This pull request removes any observable change from the proposal, because hosts would be allowed again to schedule the resolution of already loaded modules in a future macro task.

nicolo-ribaudo commented 2 years ago

While I believe that the HTML spec doesn't need this change, browsers disagree with me (https://github.com/nicolo-ribaudo/modules-import-hooks-refactor/issues/5#issuecomment-1234348207).

I'll merge this PR for now, to be sure that this refactor allows browsers to keep their current behavior. I might present a need-consensus PR in the future to avoid this unnecessary host hook call.