paulmillr / es6-shim

ECMAScript 6 compatibility shims for legacy JS engines
http://paulmillr.com
MIT License
3.11k stars 389 forks source link

Detect and override Safari 7.1 native Promises #289

Closed ljharb closed 9 years ago

ljharb commented 9 years ago

@cscott, what do you think? Would it be worth just wrapping Safari 7.1's native Promises rather than completely overriding them?

Fixes #288.

cscott commented 9 years ago

I think in this early implementation stage we should just override the promise implementation. We are already doing that on chrome I think. Note that we do already use the native Promise scheduler if available, though.

ljharb commented 9 years ago

Will we continue to use the native Promise scheduler on Safari 7.1 with this change?

cscott commented 9 years ago

I believe so. Look for makePromiseAsap in the code. --scott

                     ( http://cscott.net/ )