fix: proper detection of global in WebWorker

Author: mhevery

Diff for: lib/browser/browser.ts

@@ -9,7 +9,7 @@ import {patchMethod, patchPrototype, patchClass, zoneSymbol} from "../common/uti
 const set = 'set';
 const clear = 'clear';
 const blockingMethods = ['alert', 'prompt', 'confirm'];
-const _global = typeof window == 'undefined' ? global : window;
+const _global = typeof window === 'object' && window || typeof self === 'object' && self || global;
 
 patchTimer(_global, set, clear, 'Timeout');
 patchTimer(_global, set, clear, 'Interval');

Diff for: lib/common/utils.ts

@@ -7,7 +7,7 @@
 // Hack since TypeScript isn't compiling this for a worker.
 declare const WorkerGlobalScope;
 export const zoneSymbol: (name: string) => string = Zone['__symbol__'];
-const _global = typeof window == 'undefined' ? global : window;
+const _global = typeof window === 'object' && window || typeof self === 'object' && self || global;
 
 export function bindArguments(args: any[], source: string): any[] {
   for (let i = args.length - 1; i >= 0; i--) {

Diff for: lib/node/node.ts

@@ -3,7 +3,7 @@ import {patchTimer} from '../common/timers';
 
 const set = 'set';
 const clear = 'clear';
-const _global = typeof window === 'undefined' ? global : window;
+const _global = typeof window === 'object' && window || typeof self === 'object' && self || global;
 
 // Timers
 const timers = require('timers');

Diff for: lib/zone-spec/fake-async-test.ts

@@ -258,4 +258,4 @@
   // Export the class so that new instances can be created with proper
   // constructor params.
   Zone['FakeAsyncTestZoneSpec'] = FakeAsyncTestZoneSpec;
-})(typeof window !== 'undefined' ? window : global);
+})(typeof window === 'object' && window || typeof self === 'object' && self || global);

Diff for: lib/zone-spec/wtf.ts

@@ -137,4 +137,4 @@
   }
 
   Zone['wtfZoneSpec'] = !wtfEnabled ? null : new WtfZoneSpec();
-})(typeof window == 'undefined' ? global : window);
+})(typeof window === 'object' && window || typeof self === 'object' && self || global);

Diff for: lib/zone.ts

@@ -1157,4 +1157,4 @@ const Zone: ZoneType = (function(global: any) {
   // This is not part of public API, but it is usefull for tests, so we expose it.
   Promise[Zone.__symbol__('uncaughtPromiseErrors')] = _uncaughtPromiseErrors;
   return global.Zone = Zone;
-})(typeof window === 'undefined' ? global : window);
+})(typeof window === 'object' && window || typeof self === 'object' && self || global);

Diff for: test/zone_worker_entry_point.ts

@@ -1,6 +1,4 @@
 // Setup tests for Zone without microtask support
-(self as any).global = self;
-
 System.config({
   defaultJSExtensions: true
 });