@IgorMinar , should we also remove wtfZoneSpec from ngZone here https://github.com/angular/angular/blob/master/packages/core/src/zone/ng_zone.ts#L133?

You can preview b6b42b2 at https://pr33949-b6b42b2.ngbuilds.io/.

hmm @JiaLiPassion I didn't realize that there was a wtf specific zone spec.

Is anyone using it? this zone spec is separate from exposing the wtf apis so I think it's ok to remove stuff from core, but keep wtf support in zone.js.

thanks for pointing this out btw

I'm marking this as blocked until we agree on the zone.js spec story

You can preview 937d528 at https://pr33949-937d528.ngbuilds.io/.

looking into this more, I think we should go ahead with this change as is without removing the zone.js integration in ng_zone. That can be done separately in the future.

merge-assistance: global approval

presubmit = http://test/OCL:281862610:BASE:281862635:1574383796718:82936f1b

@IgorMinar , yeah, sure, there is no harm to leave the wtfZoneSpec in ngZone for now, and I don't think any one are really using it because wtf itself is not continued. Thanks for looking into it.

presubmit = https://tap.corp.google.com/ui#id=OCL:281862610:BASE:281985927:1574444478051:6a401beb

ivy-presubmit = https://test.corp.google.com/ui#id=OCL:281862610:BASE:281986438:1574444628677:e6a8c4d4

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}