Revert "feat: Monkey patches MessagePort onproperties (onmessage/onmessageerror) (#34610)" (#35973)

AndrewKushnir · matsko · commit 71309d223d26 · 2020-03-09T17:31:28.000-04:00
This reverts commit 1882451. Reason: breaks some g3 targets. PR Close #35973
diff --git a/packages/zone.js/MODULE.md b/packages/zone.js/MODULE.md
@@ -43,7 +43,6 @@ Below is the full list of currently supported modules.
 |PromiseRejectionEvent|PromiseRejectEvent will fire when ZoneAwarePromise has unhandled error|__Zone_disable_PromiseRejectionEvent = true|
 |mediaQuery|mediaQuery addListener API will be patched as Zone aware EventTask. (By default, mediaQuery patch will not be loaded by zone.js) |__Zone_disable_mediaQuery = true|
 |notification|notification onProperties API will be patched as Zone aware EventTask. (By default, notification patch will not be loaded by zone.js) |__Zone_disable_notification = true|
-|MessagePort|MessagePort onProperties API will be patched as Zone aware EventTask. (By default, MessagePort patch will not be loaded by zone.js) |__Zone_disable_MessagePort = true|
 
 - NodeJS
 
diff --git a/packages/zone.js/bundles.bzl b/packages/zone.js/bundles.bzl
@@ -40,7 +40,6 @@ ES5_BUNDLES = {
     "webapis-shadydom": _DIR + "browser/shadydom",
     "zone-patch-socket-io": _DIR + "extra/socket-io",
     "zone-patch-user-media": _DIR + "browser/webapis-user-media",
-    "zone-patch-message-port": _DIR + "browser/message-port",
     "zone-testing": _DIR + "testing/zone-testing",
     "zone-testing-bundle": _DIR + "browser/rollup-legacy-test-main",
 }
diff --git a/packages/zone.js/lib/browser/message-port.ts b/packages/zone.js/lib/browser/message-port.ts
diff --git a/packages/zone.js/test/browser/messageport.spec.ts b/packages/zone.js/test/browser/messageport.spec.ts
diff --git a/packages/zone.js/test/browser_entry_point.ts b/packages/zone.js/test/browser_entry_point.ts
@@ -28,4 +28,3 @@ import './browser/Worker.spec';
 import './mocha-patch.spec';
 import './jasmine-patch.spec';
 import './extra/cordova.spec';
-import './browser/messageport.spec';
diff --git a/packages/zone.js/test/karma_test.bzl b/packages/zone.js/test/karma_test.bzl
@@ -57,7 +57,6 @@ def karma_test(name, env_srcs, env_deps, env_entry_point, test_srcs, test_deps,
             "//packages/zone.js/dist:zone-patch-fetch.js",
             "//packages/zone.js/dist:zone-patch-resize-observer.js",
             "//packages/zone.js/dist:zone-patch-user-media.js",
-            "//packages/zone.js/dist:zone-patch-message-port.js",
             ":" + name + "_rollup.umd",
         ]
 
diff --git a/packages/zone.js/test/npm_package/npm_package.spec.ts b/packages/zone.js/test/npm_package/npm_package.spec.ts
@@ -123,8 +123,6 @@ describe('Zone.js npm_package', () => {
           'zone-patch-fetch.min.js',
           'zone-patch-jsonp.js',
           'zone-patch-jsonp.min.js',
-          'zone-patch-message-port.js',
-          'zone-patch-message-port.min.js',
           'zone-patch-promise-test.js',
           'zone-patch-promise-test.min.js',
           'zone-patch-resize-observer.js',

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,6 @@ ES5_BUNDLES = {`
`40`	`40`	`"webapis-shadydom": _DIR + "browser/shadydom",`
`41`	`41`	`"zone-patch-socket-io": _DIR + "extra/socket-io",`
`42`	`42`	`"zone-patch-user-media": _DIR + "browser/webapis-user-media",`
`43`		`- "zone-patch-message-port": _DIR + "browser/message-port",`
`44`	`43`	`"zone-testing": _DIR + "testing/zone-testing",`
`45`	`44`	`"zone-testing-bundle": _DIR + "browser/rollup-legacy-test-main",`
`46`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,6 @@ def karma_test(name, env_srcs, env_deps, env_entry_point, test_srcs, test_deps,`
`57`	`57`	`"//packages/zone.js/dist:zone-patch-fetch.js",`
`58`	`58`	`"//packages/zone.js/dist:zone-patch-resize-observer.js",`
`59`	`59`	`"//packages/zone.js/dist:zone-patch-user-media.js",`
`60`		`- "//packages/zone.js/dist:zone-patch-message-port.js",`
`61`	`60`	`":" + name + "_rollup.umd",`
`62`	`61`	`]`
`63`	`62`