event fired on readystatechange doesn't contain currentTarget

[ejrcavenago]

Hi!

I have a case of conflicting library. I've used xdomain for a long time, but now I'm trying to use hls.js to reproduce videos using HTTP Live Streaming.

Hls.js rely on event.currentTarget to retrieve the xhr object and continue with the code, but it seems that xhook strip that off merging with the result of fakeEvent on firing events.

[zaide-chris]

In the final js changing

    try {
      dst[k] = src[k];
    } catch (_error) {
    }

to

    try {
      dst[k] = v;
      if(dst[k] !== v){
        Object.defineProperty(dst, k, {writable: false, value: v})
      }
    } catch (_error) {
    }

and both of the facade[FIRE]("readystatechange", {}); lines
to facade[FIRE]("readystatechange", {currentTarget: facade});
Fixes this issue.

[holly-hacker]

@jpillora could this be incorporated in a new release of xhook? This issue is a blocker for me and the suggested patch fixed it.

[holly-hacker]

Seems like duplicate of #96

[jpillora]

#108 will land soon, and then any/all fixes should go in the new ES6 code base