Editorial: modernize connection prose

Author: annevk

fetch.bs

@@ -2194,10 +2194,11 @@ unset or <a for=request>keepalive</a> is false, <a lt=terminated for=fetch>termi
 <h3 id=connections>Connections</h3>
 
 <p>A user agent has an associated <dfn export id=concept-connection-pool>connection pool</dfn>. A
-<a>connection pool</a> consists of zero or more
+<a>connection pool</a> is an <a for=/>ordered set</a> of zero or more
 <dfn lt=connection export id=concept-connection>connections</dfn>. Each <a>connection</a> is
-identified by a <b>key</b> (a <a>network partition key</a>), an <b>origin</b> (an
-<a for=/>origin</a>), and <b>credentials</b> (a boolean).
+identified by an associated <dfn for=connection>key</dfn> (a <a>network partition key</a>),
+<dfn for=connection>origin</dfn> (an <a for=/>origin</a>), and <dfn for=connection>credentials</dfn>
+(a boolean).
 
 <p>Each <a>connection</a> has an associated
 <dfn for=connection id=concept-connection-timing-info>timing info</dfn> (a
@@ -2259,11 +2260,12 @@ steps:
 
 <hr>
 
-<p>To <dfn export id=concept-connection-obtain>obtain a connection</dfn>, given a <var>key</var>,
-<var>origin</var>, <var>credentials</var>, an optional boolean <var>forceNew</var> (default false),
-an optional boolean <dfn export for="obtain a connection"><var>http3Only</var></dfn> (default
-false), and an optional boolean <dfn export for="obtain a connection"><var>dedicated</var></dfn>
-(default false), run these steps:
+<p>To <dfn export id=concept-connection-obtain>obtain a connection</dfn>, given a
+<a>network partition key</a> <var>key</var>, <a for=/>origin</a>, <var>origin</var>, boolean
+<var>credentials</var>, an optional boolean <var>forceNew</var> (default false), an optional boolean
+<dfn export for="obtain a connection"><var>http3Only</var></dfn> (default false), and an optional
+boolean <dfn export for="obtain a connection"><var>dedicated</var></dfn> (default false), run these
+steps:
 <!-- http3Only and dedicated have been added for WebTransport -->
 
 <ol>
@@ -2272,8 +2274,9 @@ false), and an optional boolean <dfn export for="obtain a connection"><var>dedic
 
   <ol>
    <li><p>Let <var>connections</var> be a set of <a>connections</a> in the user agent's
-   <a>connection pool</a> each of whose <b>key</b> is <var>key</var>, <b>origin</b> is
-   <var>origin</var>, and <b>credentials</b> is <var>credentials</var>.
+   <a>connection pool</a> whose <a for=connection>key</a> is <var>key</var>,
+   <a for=connection>origin</a> is <var>origin</var>, and <a for=connection>credentials</a> is
+   <var>credentials</var>.
 
    <li><p>If <var>connections</var> is not empty and <var>http3Only</var> is false, then return
    one of <var>connections</var>.
@@ -2289,7 +2292,9 @@ false), and an optional boolean <dfn export for="obtain a connection"><var>dedic
 
   <ol>
    <li>
-    <p>Set <var>connection</var> to a new <a for=/>connection</a>.
+    <p>Set <var>connection</var> to a new <a for=/>connection</a> whose <a for=connection>key</a> is
+    <var>key</var>, <a for=connection>origin</a> is <var>origin</var>, and
+    <a for=connection>credentials</a> is <var>credentials</var>.
     <a for=/>Record connection timing info</a> given <var>connection</var> and use
     <var>connection</var> to establish an HTTP connection to <var>origin</var>.
     [[!HTTP]] [[!HTTP-SEMANTICS]] [[!HTTP-COND]] [[!HTTP-CACHING]] [[!HTTP-AUTH]] [[!TLS]]
@@ -2315,9 +2320,8 @@ false), and an optional boolean <dfn export for="obtain a connection"><var>dedic
    <li><p>Return failure.
   </ol>
 
- <li><p>If <var>dedicated</var> is false, then add <var>connection</var> to the user agent's
- <a>connection pool</a> with <b>key</b> being <var>key</var>, <b>origin</b> being <var>origin</var>,
- and <b>credentials</b> being <var>credentials</var>.
+ <li><p>If <var>dedicated</var> is false, then <a for=set>append</a> <var>connection</var> to the
+ user agent's <a>connection pool</a>.
 
  <li><p>Return <var>connection</var>.
 </ol>