Re: [whatwg/streams] Editorial: Update usage of CreateBuiltinFunction (#1140)

@TimothyGu commented on this pull request.



> @@ -5747,9 +5745,8 @@ in the constructor.
 
  1. Let |steps| be the following steps, given |chunk|:
   1. Return ? [$GetV$](|chunk|, "`byteLength`").
- 1. Let |F| be ! [$CreateBuiltinFunction$](|steps|, « », |globalObject|'s [=relevant Realm=]).
- 1. Perform ! [$SetFunctionName$](|F|, "`size`").
- 1. Perform ! [$SetFunctionLength$](|F|, 1).
+ 1. Let |F| be ! [$CreateBuiltinFunction$](|steps|, 1, "`size`", « », |globalObject|'s [=relevant
+    Realm=]).

Actually sorry, this is technically a normative change as it changes the enumeration order of `name` vs `length`, but Chrome and Firefox both already conform to this change (`length` comes before `name).

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