Skip to content

Commit 2e46596

Browse files
JeanMechethePunderWoman
authored andcommitted
refactor(zone.js): remove legacy browser support (#63511)
This commit removes the support for legacy browsers. BREAKING CHANGE: IE/Non-Chromium Edge are not supported anymore. PR Close#63511
1 parent df780fa commit 2e46596

25 files changed

Lines changed: 28 additions & 1086 deletions

‎packages/zone.js/MODULE.md‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ Below is the full list of currently supported modules.
4040
| IntersectionObserver | Intersection will be patched as Zone aware operation |\_\_Zone_disable_IntersectionObserver = true |
4141
| FileReader | FileReader will be patched as Zone aware operation |\_\_Zone_disable_FileReader = true |
4242
| canvas | HTMLCanvasElement.toBlob will be patched as Zone aware operation |\_\_Zone_disable_canvas = true |
43-
| IE BrowserTools check | in IE, browser tool will not use zone patched eventListener |\_\_Zone_disable_IE_check = true |
4443
| CrossContext check | in webdriver, enable check event listener is cross context |\_\_Zone_enable_cross_context_check = true |
4544
|`beforeunload`| enable the default `beforeunload` handling behavior, where event handlers return strings to prompt the user |**zone_symbol**enable_beforeunload = true |
4645
| XHR | XMLHttpRequest will be patched as Zone aware MacroTask |\_\_Zone_disable_XHR = true |

‎packages/zone.js/bundles.bzl‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,6 @@ BUNDLES_ENTRY_POINTS = {
5555
"zone-error": {
5656
"entrypoint": _DIR+"common/rollup-error-rewrite",
5757
},
58-
"zone-legacy": {
59-
"entrypoint": _DIR+"browser/rollup-browser-legacy",
60-
},
6158
"zone-bluebird": {
6259
"entrypoint": _DIR+"extra/rollup-bluebird",
6360
},

‎packages/zone.js/karma-dist-sauce-jasmine3.conf.js‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ module.exports = function (config) {
1818
'SL_IOS8',
1919
'SL_IOS9',
2020
'SL_IOS10',
21-
'SL_MSEDGE15',
2221
'SL_ANDROID4.4',
2322
'SL_ANDROID5.1',
2423
]);

‎packages/zone.js/lib/BUILD.bazel‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ exports_files(glob([
99
copy_to_bin(
1010
name="files",
1111
srcs= [
12-
"browser/rollup-browser-legacy.ts",
1312
"browser/rollup-canvas.ts",
1413
"browser/rollup-message-port.ts",
1514
"browser/rollup-shadydom.ts",

‎packages/zone.js/lib/browser/api-util.ts‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import {
1919
bindArguments,
2020
FALSE_STR,
2121
isBrowser,
22-
isIEOrEdge,
2322
isMix,
2423
isNode,
2524
ObjectCreate,
@@ -64,7 +63,6 @@ export function patchUtil(Zone: ZoneType): void {
6463
}
6564
api.patchEventPrototype=patchEventPrototype;
6665
api.patchEventTarget=patchEventTarget;
67-
api.isIEOrEdge=isIEOrEdge;
6866
api.ObjectDefineProperty=ObjectDefineProperty;
6967
api.ObjectGetOwnPropertyDescriptor=ObjectGetOwnPropertyDescriptor;
7068
api.ObjectCreate=ObjectCreate;

‎packages/zone.js/lib/browser/browser-legacy.ts‎

Lines changed: 0 additions & 46 deletions
This file was deleted.

‎packages/zone.js/lib/browser/browser.ts‎

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,6 @@ import {eventTargetPatch, patchEvent} from './event-target';
2929
import{propertyDescriptorPatch}from'./property-descriptor';
3030

3131
exportfunctionpatchBrowser(Zone: ZoneType): void{
32-
Zone.__load_patch('legacy',(global: any)=>{
33-
constlegacyPatch=global[Zone.__symbol__('legacyPatch')];
34-
if(legacyPatch){
35-
legacyPatch();
36-
}
37-
});
38-
3932
Zone.__load_patch('timers',(global: any)=>{
4033
constset='set';
4134
constclear='clear';

‎packages/zone.js/lib/browser/event-target-legacy.ts‎

Lines changed: 0 additions & 133 deletions
This file was deleted.

0 commit comments

Comments
 (0)