Revert "timers: refactor to use optional chaining"

This reverts commit d8f535b. PR-URL: #38245 Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zijian Liu <[email protected]> Reviewed-By: Beth Griggs <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Rich Trott <[email protected]>

Author: mcollina

lib/timers.js

@@ -171,7 +171,7 @@ ObjectDefineProperty(setTimeout, customPromisify,{
 });
 
 functionclearTimeout(timer){
-if(timer?._onTimeout){
+if(timer&&timer._onTimeout){
 timer._onTimeout=null;
 unenroll(timer);
 return;

lib/timers/promises.js

@@ -53,7 +53,10 @@ function setTimeout(after, value, options ={}){
 'boolean',
 ref));
 }
-if(signal?.aborted){
+// TODO(@jasnell): If a decision is made that this cannot be backported
+// to 12.x, then this can be converted to use optional chaining to
+// simplify the check.
+if(signal&&signal.aborted){
 returnPromiseReject(newAbortError());
 }
 letoncancel;
@@ -95,7 +98,10 @@ function setImmediate(value, options ={}){
 'boolean',
 ref));
 }
-if(signal?.aborted){
+// TODO(@jasnell): If a decision is made that this cannot be backported
+// to 12.x, then this can be converted to use optional chaining to
+// simplify the check.
+if(signal&&signal.aborted){
 returnPromiseReject(newAbortError());
 }
 letoncancel;