Patch all browsers API returning Promise

[vicb]

All browsers API returning Promise should be patched.

This is required because the returned Promises would not extend the monkey-patched Promise by default.

It seems like patching might not be needed when microtasks are not used (the monkey patch on the Promise.prototype is taken into account on latest Chrome version (44) but it fails on earlier version. It should probably be tested when other browsers implement the API before we remove the patch on zone.js - we must keep it for zone-microtask.js)

ref:

• fetch() has been patched in feat(Change Detection): Implement collection changes #119
• idl can be download via the script in feat(transpiler): map type declarations to dart. #118

[alxhub]

FYI, I just ran into this using the PushManager APIs, which return Promises that don't extend the zone-patched version and thus escape the current zone when used.

[maliksahil]

This is causing a lot of headache. I wish we could use promises or apis that use promises without having to think about it.

[vicb]

@alxhub could you please submit a PR for your use case.

@maliksahil what API would you like to see patched ? PR are welcome.

[AndrewKushnir]

Closing this ticket since ZoneJS already covers a large subset of APIs and we can create new tickets in case additional APIs should be patched.

[angular-automatic-lock-bot]

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.}