Closed Bug 581018 Opened 14 years ago Closed 14 years ago

random failure in docshell test_bug293235.xul | A pageshow event was expected for page Bug 293235 page1 ...

Categories

(Toolkit :: Places, defect)

Product:
Toolkit
Component:
Places
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla2.0b3

People

(Reporter: mak, Assigned: mak)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

patch v1.0
1.49 KB, patch
sdwilsh
: review+
Details | Diff | Splinter Review 
this has started with removal of lazy add, need to better check observers there.

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1279805988.1279807633.32320.gz

1322 INFO TEST-START | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul
TEST: registering a listener for pageshow events
TEST: eventListener received a pageshow event for page Bug 293235 page1, persisted=false
TEST: removing event listeners
1323 INFO TEST-PASS | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | Unexpected topic - "visited-status-resolution" should equal "visited-status-resolution"
1324 INFO TEST-PASS | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | references should not match
1325 INFO TEST-PASS | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | link should initially be blue
TEST: registering a listener for pageshow events
TEST: registering a listener for pageshow events
TEST: eventListener received a pageshow event for page , persisted=false
1326 INFO TEST-PASS | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | A pageshow event was expected, but a pageshow event occurred - "pageshow" should equal "pageshow"
1327 INFO TEST-PASS | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | originalTarget for last pageshow event not an HTMLDocument
NEXT ERROR 1328 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | A pageshow event was expected for page Bug 293235 page1, but was fired for page  - got "", expected "Bug 293235 page1"
1329 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | The persisted property of the pageshowevent on page  had an unexpected value - got false, expected true
TEST: eventListener received a pageshow event for page Bug 293235 page1, persisted=true
NEXT ERROR 1330 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | Unexpected event (pageshow) occurred
TEST: removing event listeners
1331 INFO TEST-PASS | chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul | visited link should be purple
1332 INFO SimpleTest finished chrome://mochikit/content/chrome/docshell/test/chrome/test_bug293235.xul in 1406ms
Attached patch patch v1.0 — — Splinter Review 
there is no way to tell if we'll get first pageshow or visit, thus we should wait both.
Attachment #459605 - Flags: review?(sdwilsh)
Comment on attachment 459605 [details] [diff] [review]
patch v1.0

r=sdwilsh
Attachment #459605 - Flags: review?(sdwilsh) → review+
http://hg.mozilla.org/mozilla-central/rev/9f94b4f80463
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b3
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: