Bug Tracker

#14379 closed bug (fixed)

Opened September 20, 2013 06:55PM UTC

Closed October 28, 2013 09:54PM UTC

Last modified November 15, 2013 05:50PM UTC

Issue with xhr.js

Reported by: Sabran Owned by: gibson042
Priority: undecided Milestone: 1.11/2.1
Component: ajax Version: 1.10.2
Keywords: Cc:
Blocked by: Blocking:
Description

I’m trying to use the latest version of jQuery and the latest version of jquery.signalR. I’m running into a problem in IE9 specifically. On window.unload, signalR is firing off an AJAX request. This fails in jquery with the error: Unable to set value of the property 'x': object is null or undefined. The ‘x’ is actually a number, but it varies.

I’ve traced the issue down to line 104 in xhr.js:

callback = xhrCallbacks[( id = xhrId++ )] = callback("abort");

The problem is that xhrCallbacks is undefined at the time of this call. The reason it is undefined is because earlier, on line 32, it is explicitly set to undefined on the window.unload event.

Attachments (0)
Change History (10)

Changed September 23, 2013 03:07PM UTC by Sabran comment:1

Since this is a problem related to window.unload being called, I don't think it's reproducible using jsFiddle. Sorry.

Changed September 23, 2013 07:41PM UTC by gibson042 comment:2

owner: → Sabran
status: new → pending

Replying to [comment:1 Sabran]:

Since this is a problem related to window.unload being called, I don't think it's reproducible using jsFiddle. Sorry.

I recommend a reduced test case that either manually triggers "unload" or leverages iframes.

Changed September 23, 2013 09:23PM UTC by Sabran comment:3

status: pending → new

Ok. Here's the jsFiddle which manually triggers unload:

http://jsfiddle.net/wE3YU/3/

Thanks!

Changed September 23, 2013 09:28PM UTC by Sabran comment:4

Just a reminder, I'm only able to reproduce the error in IE9. It seems to work fine in Chrome and Firefox.

Changed September 26, 2013 03:07PM UTC by Sabran comment:5

Is there anything else needed from me to verify that this is a legitimate issue with jQuery?

Changed September 26, 2013 09:21PM UTC by gibson042 comment:6

owner: Sabran → gibson042
status: new → assigned

No; that'll do. Thanks!

Changed October 17, 2013 06:29PM UTC by anonymous comment:7

Any ETA on when this might get prioritized?

Changed October 21, 2013 04:16PM UTC by gibson042 comment:8

milestone: None → 1.11/2.1

Changed October 28, 2013 09:54PM UTC by Richard Gibson comment:9

resolution: → fixed
status: assigned → closed

Fix #14379: AJAX requests on unload

Changeset: f9d41ac641dcb5a93ba8a9027476b160d8f41111

Changed November 15, 2013 05:50PM UTC by dmethvin comment:10

component: unfiled → ajax