Closed Bug 1058072 Opened 10 years ago Closed 10 years ago

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

Categories

(Core :: DOM: Events, defect)

Product:
Core
Component:
DOM: Events
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla34
Tracking Flags:
Tracking Status
firefox32 --- wontfix
firefox33 --- fixed
firefox34 --- fixed
firefox-esr24 --- unaffected
firefox-esr31 --- fixed
b2g-v1.4 --- unaffected
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

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

References

(
URL
)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

1058072.diff
1.61 KB, patch
smaug
: review+
Details | Diff | Splinter Review
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()
Attached patch 1058072.diff — — Splinter 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+
Keywords: checkin-needed
Attachment #8478328 - Attachment filename: 1058072.diff → 1058072.diff (for check-in)
https://hg.mozilla.org/mozilla-central/rev/4c79080181fd
Status: NEW → RESOLVED
Closed: 10 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.

Attachment

General

Creator:
Created:
Updated:
Size: