Revert "fix(zone.js): enable monkey patching of the `queueMicrotask()…

…` API in node.js (#50467)" (#50529) This reverts commit 381cb98. PR Close #50529
angular · May 31, 2023 · 7567348 · 7567348
1 parent 069498c
commit 7567348
Show file tree

Hide file tree

Showing 11 changed files with 12 additions and 27 deletions.
diff --git a/packages/zone.js/karma-build.conf.js b/packages/zone.js/karma-build.conf.js
@@ -13,6 +13,5 @@ module.exports = function(config) {
   config.files.push('build/test/test_fake_polyfill.js');
   config.files.push('build/lib/zone.js');
   config.files.push('build/lib/common/promise.js');
-  config.files.push('build/lib/common/queue-microtask.js');
   config.files.push('build/test/main.js');
 };
diff --git a/packages/zone.js/lib/browser/browser.ts b/packages/zone.js/lib/browser/browser.ts
@@ -25,6 +25,15 @@ Zone.__load_patch('legacy', (global: any) => {
   }
 });
 
+Zone.__load_patch('queueMicrotask', (global: any, Zone: ZoneType, api: _ZonePrivate) => {
+  api.patchMethod(global, 'queueMicrotask', delegate => {
+    return function(self: any, args: any[]) {
+      Zone.current.scheduleMicroTask('queueMicrotask', args[0]);
+    }
+  });
+});
+
+
 Zone.__load_patch('timers', (global: any) => {
   const set = 'set';
   const clear = 'clear';

diff --git a/packages/zone.js/lib/browser/rollup-common.ts b/packages/zone.js/lib/browser/rollup-common.ts
@@ -10,4 +10,3 @@ import '../zone';
 import '../common/promise';
 import '../common/to-string';
 import './api-util';
-import '../common/queue-microtask';
diff --git a/packages/zone.js/lib/common/queue-microtask.ts b/packages/zone.js/lib/common/queue-microtask.ts
diff --git a/packages/zone.js/lib/node/node.ts b/packages/zone.js/lib/node/node.ts
@@ -9,7 +9,6 @@
 import './node_util';
 import './events';
 import './fs';
-import '../common/queue-microtask';
 
 import {findEventTasks} from '../common/events';
 import {patchTimer} from '../common/timers';

diff --git a/packages/zone.js/lib/zone.configurations.api.ts b/packages/zone.js/lib/zone.configurations.api.ts
@@ -296,9 +296,9 @@ interface ZoneGlobalConfigurations {
 
   /**
    *
-   * Disable the monkey patching of the `queueMicrotask()` API.
+   * Disable the monkey patching of the browser's `queueMicrotask()` API.
    *
-   * By default, `zone.js` monkey patches the `queueMicrotask()` API
+   * By default, `zone.js` monkey patches the browser's `queueMicrotask()` API
    * to ensure that `queueMicrotask()` callback is invoked in the same zone as zone used to invoke
    * `queueMicrotask()`. And also the callback is running as `microTask` like
    * `Promise.prototype.then()`.

diff --git a/packages/zone.js/test/browser-zone-setup.ts b/packages/zone.js/test/browser-zone-setup.ts
@@ -11,7 +11,6 @@ if (typeof window !== 'undefined') {
   (window as any)[zoneSymbol('fakeAsyncAutoFakeAsyncWhenClockPatched')] = true;
 }
 import '../lib/common/to-string';
-import '../lib/common/queue-microtask';
 import '../lib/browser/api-util';
 import '../lib/browser/browser-legacy';
 import '../lib/browser/browser';

diff --git a/...ne.js/test/common/queue-microtask.spec.ts → ...e.js/test/browser/queue-microtask.spec.ts b/...ne.js/test/common/queue-microtask.spec.ts → ...e.js/test/browser/queue-microtask.spec.ts
diff --git a/packages/zone.js/test/browser_entry_point.ts b/packages/zone.js/test/browser_entry_point.ts
@@ -28,3 +28,4 @@ import './mocha-patch.spec';
 import './jasmine-patch.spec';
 import './browser/messageport.spec';
 import './extra/cordova.spec';
+import './browser/queue-microtask.spec';
diff --git a/packages/zone.js/test/common_tests.ts b/packages/zone.js/test/common_tests.ts
@@ -11,7 +11,6 @@ import './common/zone.spec';
 import './common/task.spec';
 import './common/util.spec';
 import './common/Promise.spec';
-import './common/queue-microtask.spec';
 import './common/fetch.spec';
 import './common/Error.spec';
 import './common/setInterval.spec';

diff --git a/packages/zone.js/test/node_error_entry_point.init.ts b/packages/zone.js/test/node_error_entry_point.init.ts
@@ -11,7 +11,6 @@ import './test_fake_polyfill';
 // Setup tests for Zone without microtask support
 import '../lib/zone';
 import '../lib/common/promise';
-import '../lib/common/queue-microtask';
 import '../lib/common/to-string';
 import './test-env-setup-jasmine';
 import './wtf_mock';