Opened 11 years ago

Closed 11 years ago

#9448 closed defect (fixed)

test_placeStrict hard locks Safari 4 from file:///

Reported by: haysmark Owned by: bill
Priority: high Milestone: 1.3.2
Component: Dijit Version: 1.3.1
Keywords: Cc:
Blocked By: Blocking:

Description

Load dijit/tests/_base/test_placeStrict.html from ​file:/// with Safari 4 for Windows. The status bar's progress stops ~45 of 66 items and although the spinning loading icon keep going, the browser becomes unresponsive. This only seems to happen from ​file:///.

Attachments (1)

9448.patch​ (571 bytes) - added by haysmark 11 years ago.
Fixes #9448. Change alert to console.log to prevent test from crashing in Safari 4.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by haysmark

Attachment: 9448.patch​ added

Fixes #9448. Change alert to console.log to prevent test from crashing in Safari 4.

comment:1 Changed 11 years ago by haysmark

Milestone: tbd → 1.3.2

Bill, I attached a fix to Safari locking up in the test_placeStrict test. It seems to be a timing issue with Safari related to the alert happening before the page renders. Using console.log in place of alert fixes it.

comment:2 Changed 11 years ago by bill

Oh I see, it's just a test problem. Actually test_placeStrict.html is no longer needed; viewport.html does the same test automatically. I'll just remove it.

comment:3 Changed 11 years ago by bill

Resolution: → fixed
Status: new → closed

(In [18535]) Remove manual tests of getViewport() in favor of automated ones, and remove duplicated code for quirks mode vs. standards mode tests.

Fixes #9448.

Note: See TracTickets for help on using tickets.