Skip to content

Editorial: fix accidental links to the URL concept #11916

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Editorial: fix accidental links to the URL concept #11916

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions source
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -130988,10 +130988,11 @@ interface <dfn interface>SharedWorkerGlobalScope</dfn> : <span>WorkerGlobalScope

<li><p>Set <var>settings object</var>'s <span data-x="concept-environment-id">id</span> to a new
unique opaque string, <span data-x="concept-environment-creation-url">creation URL</span> to
<var>worker global scope</var>'s <span>url</span>, <span>top-level creation URL</span> to null,
<span data-x="concept-environment-target-browsing-context">target browsing context</span> to
null, and <span data-x="concept-environment-active-service-worker">active service worker</span>
to null.</p></li>
<var>worker global scope</var>'s <span data-x="concept-WorkerGlobalScope-url">url</span>,
<span>top-level creation URL</span> to null, <span
data-x="concept-environment-target-browsing-context">target browsing context</span> to null, and
<span data-x="concept-environment-active-service-worker">active service worker</span> to
null.</p></li>

<li><p>If <var>worker global scope</var> is a <code>DedicatedWorkerGlobalScope</code> object,
then set <var>settings object</var>'s <span>top-level origin</span> to <var>outside
Expand Down Expand Up @@ -143491,8 +143492,8 @@ document.body.appendChild(text);

<li><p>Let <var>url</var> be the <span>URL</span> that <var>element</var> would fetch if it was
processed normally. If there is no such <span>URL</span> or if it is the empty string, then do
nothing. Otherwise, if <span>url</span> is already in the <span>list of speculative fetch
URLs</span>, then do nothing. Otherwise, fetch <span>url</span> as if the element was processed
nothing. Otherwise, if <var>url</var> is already in the <span>list of speculative fetch
URLs</span>, then do nothing. Otherwise, fetch <var>url</var> as if the element was processed
Copy link
Member

@annevk annevk Nov 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems a bit wonky given that this is the same paragraph we initialize url in, but it might be correct? Ideally we'd redo this in multiple steps to make it less ambiguous.

cc @zcorpan

normally, and add <var>url</var> to the <span>list of speculative fetch URLs</span>.</p></li>
</ul>
</div>
Expand Down