search

stefanpenner / ember-browsery-stats

MIT License
60 stars 7 forks source link

Incompatibility with mobile safari (iOS) #8

Open mike-north opened 9 years ago

mike-north commented 9 years ago

Not sure if you were already aware of this, but there is no window.performance on mobile safari

https://github.com/stefanpenner/ember-browsery-stats/blob/7c12be10812f8e29ab3dd54102b3ae80942bd944/vendor/memory-stats.js#L53

Not sure if you'd rather investigate a polyfill (if it's even possible to instrument accurately) or maybe just a note in the readme screen shot 2015-06-20 at 4 29 24 pm

stefanpenner commented 9 years ago

I don't think this is something we can polyfil. It shouldn't crash though, rather gracefully degrade.

mike-north commented 9 years ago

agree. I was running into sauce labs failures, which should be avoided if possible

stefanpenner commented 9 years ago

seems like a bug upstream: https://github.com/paulirish/memory-stats.js/blob/master/memory-stats.js#L66-L68