Re: [whatwg/fetch] Editorial: add <div algorithm> wrappers to Header class (PR #1596)

@dlrobertson commented on this pull request.

Thanks for the review! I'll try to update this evening.

>  
-<p>The <dfn export for=Headers method><code>delete(<var>name</var>)</code></dfn> method steps are:
+<div algorithm>
+<p>The <dfn export for=Headers method><code>delete(<var>name</var>)</code></dfn> method
+from a {{Headers}} object (<var>headers</var>), run these steps:

Ah! Good point. Originally I thought this would be like [remove privileged no-CORS request-headers](https://github.com/whatwg/fetch/pull/1596/files#diff-b4827ee42aceaf9dc3e7216f72262d81c32217e9996fb70a657f259e3ac92604L6647), but it makes sense that we'd use `this` for these methods.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1596#pullrequestreview-1269864676
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1596/review/1269864676@github.com>

Received on Wednesday, 25 January 2023 18:04:41 UTC