Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Blob support detection logics #987

Merged
merged 1 commit into from Feb 29, 2016

Conversation

nelsonsilva
Copy link
Contributor

Current Blob support detection returns false in Safari 9 for example where typeof Blob is object.
This change is based on detection logics found in Modernizr.

@fatso83
Copy link
Contributor

fatso83 commented Feb 12, 2016

Seems legit to me.

@jondelga
Copy link

This also fixes tests run on phantom.

fatso83 added a commit that referenced this pull request Feb 29, 2016
Improve Blob support detection logics
@fatso83 fatso83 merged commit 9d91091 into sinonjs:master Feb 29, 2016
@pcardune
Copy link

@fatso83 Any chance this change could go into a 1.17.4 release?

@fatso83
Copy link
Contributor

fatso83 commented Apr 28, 2016

@pcardune: sure. Just supply a PR and the likelihood is quite high :-)

@pcardune
Copy link

pcardune commented May 2, 2016

@fatso83 I can do that. What steps do you follow for cutting new releases? Seeing as master is well on it's way to a 2.0 version, should I just submit a pull request against the v1.17 branch with just this change? I assume after merging, you'll go ahead and bump the version number in package.json and publish a new build to npm?

@fatso83
Copy link
Contributor

fatso83 commented May 2, 2016

@pcardune Your assumptions are correct. I'll go ahead and merge it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants