Result logged after SimpleTest.finish() with test_bug432698.html

RESOLVED FIXED in Firefox 33

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: martijn.martijn, Assigned: martijn.martijn)

Tracking

({intermittent-failure})

Trunk
mozilla34
x86
macOS
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox32 wontfix, firefox33 fixed, firefox34 fixed, firefox-esr24 unaffected, firefox-esr31 fixed, b2g-v1.4 unaffected, b2g-v2.0 fixed, b2g-v2.1 fixed)

Details

()

Attachments

(1 attachment)

Assignee

Description

5 years ago
This has become visible now that bug 1032878 has been fixed, which disallows logging of subresults after SimpleTest.finish().

https://tbpl.mozilla.org/php/getParsedLog.php?id=46688150&tree=Mozilla-Inbound
3968 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | mouseleave should not bubble! - Result logged after SimpleTest.finish()
3969 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | mouseleave is not cancelable! - Result logged after SimpleTest.finish()
3970 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | Wrong event target! - Result logged after SimpleTest.finish()
3971 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | Leaking nodes to another document? - Result logged after SimpleTest.finish()
3972 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | mouseenter should not bubble! - Result logged after SimpleTest.finish()
3973 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | mouseenter is not cancelable! - Result logged after SimpleTest.finish()
3974 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | Wrong event target! - Result logged after SimpleTest.finish()
3975 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug432698.html | Leaking nodes to another document? - Result logged after SimpleTest.finish()
Assignee

Comment 1

5 years ago
Posted patch 1058072.diffSplinter Review
This nulls out all the event handlers after the tests have been run, that fixes the logging of results after SimpleTest.finish().
Attachment #8478328 - Flags: review?(bugs)
Attachment #8478328 - Flags: review?(bugs) → review+
Assignee

Updated

5 years ago
Keywords: checkin-needed
Assignee

Updated

5 years ago
Attachment #8478328 - Attachment filename: 1058072.diff → 1058072.diff (for check-in)
https://hg.mozilla.org/mozilla-central/rev/4c79080181fd
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.