Def. in this case we need to only enable that garbage collection if attachEvent is present —
Sent from Mailbox for iPhone

On Tue, Jan 21, 2014 at 4:57 PM, Roman Shtylman <notifications@github.com>
wrote:

>>      for (var i in Request.requests) {
>>        if (Request.requests.hasOwnProperty(i)) {
>>          Request.requests[i].abort();
>>        }
>>      }
>> -  });
>> +  };
>> +
>> +  if (global.addEventListener) {
>> +    global.addEventListener('unload', unloadHandler);
> Something was triggering this in the IE 10 webworker test. We might need to
> add tests to check if these things didn't run if we didn't want them to.
> On Jan 21, 2014 7:45 PM, "Guillermo Rauch" <notifications@github.com> wrote:
>> In lib/transports/polling-xhr.js:
>>
>> >      for (var i in Request.requests) {
>> >        if (Request.requests.hasOwnProperty(i)) {
>> >          Request.requests[i].abort();
>> >        }
>> >      }
>> > -  });
>> > +  };
>> > +
>> > +  if (global.addEventListener) {
>> > +    global.addEventListener('unload', unloadHandler);
>>
>> This unload code should actually be only needed for old ie.
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/LearnBoost/engine.io-client/pull/237/files#r9061508>
>> .
>>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/LearnBoost/engine.io-client/pull/237/files#r9061765