Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#11887 closed defect (fixed)

[regression] "Stack overflow" in refreshOnShow in IE6

Reported by: haysmark Owned by: Rawld Gill
Priority: high Milestone: 1.6
Component: Loader Version: 1.5
Keywords: Cc: Katie Vance, Kris Zyp, Rawld Gill
Blocked By: Blocking:

Description

See:

​http://archive.dojotoolkit.org/dojo-2010-10-21/checkout/dijit/tests/layout/test_refreshOnShow.html

In IE6, you get an error, "Stack Overflow on line: 0". Compare to the old loader API:

​http://archive.dojotoolkit.org/dojo-2010-10-20/checkout/dijit/tests/layout/test_refreshOnShow.html

Change History (7)

comment:1 Changed 10 years ago by haysmark

Priority: normal → high
severity: normal → major

comment:2 Changed 10 years ago by Rawld Gill

Status: new → assigned

comment:3 Changed 10 years ago by Rawld Gill

This is likely consequent to merge errors made in the first couple of AMD refactor commits. Please grab a version of the code on or after [23059] and reattempt on IE6 (I would do this, but I don't have and IE6 environment available presently).

comment:4 Changed 10 years ago by haysmark

As of [23064] in test_refreshOnShow I still get Stack Overflow in IE6.

comment:5 Changed 10 years ago by bill

Cc: Katie Vance Kris Zyp Rawld Gill added
Milestone: tbd → 1.6

[23071] merged test_refreshOnShow.html into test_AccordionContainer.html and added an automated test. Both are working for me now.

IIRC this ticket was fixed by [23073] and [23117], should we close it?

comment:6 Changed 10 years ago by Kris Zyp

Resolution: → fixed
Status: assigned → closed

Yes

comment:7 Changed 10 years ago by bill

Component: General → Loader
Note: See TracTickets for help on using tickets.