Revert "DevExtreme widgets should subscribe to native events outside NgZone (#618)" (#642)

This reverts commit fd127c2.

Author: dxvladislavvolkov

src/core/events-strategy.ts

@@ -1,6 +1,5 @@
-import { EventEmitter, NgZone, Injectable } from '@angular/core';
+import { EventEmitter, NgZone } from '@angular/core';
 import { DxComponent } from './component';
-import * as eventsEngine from 'devextreme/events/core/events_engine';
 
 const dxToNgEventNames = {};
 
@@ -115,16 +114,3 @@ export class EmitterHelper {
         events.push({ name: name, context: this });
     }
 }
-
-@Injectable()
-export class EventsRegistrator {
-    constructor(ngZone: NgZone) {
-        eventsEngine.set({
-            on: function(...args) {
-                ngZone.runOutsideAngular(() => {
-                    this.callBase.apply(this, args);
-                });
-            }
-        });
-    }
-}

templates/component.tst

@@ -16,7 +16,6 @@ import {
     Input,
     Output,
     OnDestroy,
-    Injector,
     EventEmitter<#? it.isEditor #>,
     AfterContentInit,
     ContentChild,
@@ -44,7 +43,6 @@ import {
 import { <#= baseClass #> } from '../core/component';
 import { DxTemplateHost } from '../core/template-host';
 import { DxTemplateModule } from '../core/template';
-import { EventsRegistrator } from '../core/events-strategy';
 import { NestedOptionHost } from '../core/nested-option';
 import { WatcherHelper } from '../core/watcher-helper';
 <#? collectionProperties.length #>import { IterableDifferHelper } from '../core/iterable-differ-helper';<#?#>
@@ -117,12 +115,11 @@ export class <#= it.className #>Component extends <#= baseClass #> <#? implement
     }
 <#~#>
 
-    constructor(elementRef: ElementRef, ngZone: NgZone, templateHost: DxTemplateHost, injector: Injector,
+    constructor(elementRef: ElementRef, ngZone: NgZone, templateHost: DxTemplateHost,
             <#? collectionProperties.length #>private <#?#>_watcherHelper: WatcherHelper<#? collectionProperties.length #>,
             private _idh: IterableDifferHelper<#?#>, optionHost: NestedOptionHost) {
 
         super(elementRef, ngZone, templateHost, _watcherHelper);
-        injector.get(EventsRegistrator);
 
         this._createEventEmitters([
             <#~ it.events :event:i #>{ <#? event.subscribe #>subscribe: '<#= event.subscribe #>', <#?#>emit: '<#= event.emit #>' }<#? i < it.events.length-1 #>,
@@ -201,6 +198,5 @@ export class <#= it.className #>Component extends <#= baseClass #> <#? implement
     <#= component.className #>Module<#~#>,
     DxTemplateModule
   ],
-  providers: [EventsRegistrator]
 })
 export class <#= it.className #>Module { }

tests/src/ui/list.spec.ts

@@ -5,8 +5,7 @@ import {
     Component,
     ViewChildren,
     QueryList,
-    OnDestroy,
-    AfterViewChecked
+    OnDestroy
 } from '@angular/core';
 
 import {
@@ -27,15 +26,13 @@ import {
     selector: 'test-container-component',
     template: ''
 })
-class TestContainerComponent implements AfterViewChecked {
+class TestContainerComponent {
     emptyItems = undefined;
     items = [1];
     complexItems = [{ text: 'Item 1' }];
     defaultTemplateItems = [{ text: 'test', disabled: false }];
     disabled = false;
     @ViewChildren(DxListComponent) innerWidgets: QueryList<DxListComponent>;
-
-    ngAfterViewChecked() {}
 }
 
 describe('DxList', () => {
@@ -562,25 +559,4 @@ describe('DxList', () => {
         expect(DxButton['getInstance'](elements[1])).not.toBeUndefined();
     }));
 
-    it('widget events should subscribe on native events outside NgZone', () => {
-        TestBed.overrideComponent(TestContainerComponent, {
-            set: {
-                template: `<dx-list [items]="items"></dx-list>`
-            }
-        });
-
-        let fixture = TestBed.createComponent(TestContainerComponent);
-        fixture.autoDetectChanges();
-
-        let instance = getWidget(fixture);
-        let onChangesSpy = spyOn(fixture.componentInstance, 'ngAfterViewChecked');
-
-        expect(onChangesSpy.calls.count()).toBe(0);
-
-        let item = instance.element().querySelector('.dx-item');
-        item.click();
-
-        expect(onChangesSpy.calls.count()).toBe(0);
-        fixture.autoDetectChanges(false);
-    });
 });