loose instanceof: check constructor function name instead

[weilu]

#287 fixed and rebased on top of #288. Tested minified version with testling (minified with browserify test/*.js | node_modules/uglify-js/bin/uglifyjs > test.js)

[coveralls]

Coverage remained the same when pulling eb4e888 on weilu:loose-instanceof into ca09849 on bitcoinjs:master.

[dcousens]

Why not just use value.constructor.name === type.name?

[weilu]

https://kangax.github.io/compat-table/es6/#function_name_property

[dcousens]

Well, in IE, they'll both be undefined, so it won't fail... haha.

edit: I'd rather have an alternative path in there just for IE, then use a weird contorted way by default.

[dcousens]

Also, you're using type.name already... how is that any better?

[weilu]

I'd rather have an alternative path in there just for IE, then use a weird contorted way by default.

Unfortunately the "weird contorted way" is the ES5 way.

[coveralls]

Coverage increased (+0.0%) when pulling b55b10c on weilu:loose-instanceof into ca09849 on bitcoinjs:master.