lib: update class fields TODO in abort_controller.js

https://bugs.chromium.org/p/v8/issues/detail?id=10704 is already fixed, but since AbortController currently throws ERR_INVALID_THIS we'll revert to class fields in a subsequent patch. For now just update the comments. PR-URL: nodejs#42361 Refs: nodejs@b1c3909 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
xtx1130 · Apr 25, 2022 · a5e4f3e · a5e4f3e
1 parent 840f42d
commit a5e4f3e
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/internal/abort_controller.js b/lib/internal/abort_controller.js
@@ -291,9 +291,8 @@ function abortSignal(signal, reason) {
   signal.dispatchEvent(event);
 }
 
-// TODO(joyeecheung): V8 snapshot does not support instance member
-// initializers for now:
-// https://bugs.chromium.org/p/v8/issues/detail?id=10704
+// TODO(joyeecheung): use private fields and we'll get invalid access
+// validation from V8 instead of throwing ERR_INVALID_THIS ourselves.
 const kSignal = Symbol('signal');
 
 function validateAbortController(obj) {