search

freedomjs / freedom

Embracing a distributed web
http://freedomjs.org
Apache License 2.0
512 stars 53 forks source link

Add support for sending Blobs to core.xhr. #303

Closed bemasc closed 8 years ago

agallant commented 8 years ago

Test failures look unrelated (filed #304). This change itself LGTM, thanks.

agallant commented 8 years ago

Actually belay that, locally at least one of the failing tests does have to do with blobs. Still seems like it should be a coincidence but want to fix it first.

agallant commented 8 years ago

Figured it out and it is unrelated (phantomjs >= 2.0 now supports blobs, but the old fill check broke it). So yes, this pull request :+1:.