Closed Bug 757722 Opened 13 years ago Closed 13 years ago

unit_ipc/test_content_annotation.js fails on OS X 10.7 and 10.8

Categories

(Toolkit :: Crash Reporting, defect)

x86_64
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18
Tracking Status
firefox15 --- wontfix
firefox16 --- fixed
firefox17 --- fixed
firefox-esr10 --- unaffected

People

(Reporter: philor, Assigned: philor)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file)

Opt: TEST-INFO | /Users/cltbld/talos-slave/test/build/xpcshell/tests/toolkit/crashreporter/test/unit_ipc/test_content_annotation.js | running test ... TEST-UNEXPECTED-FAIL | /Users/cltbld/talos-slave/test/build/xpcshell/tests/toolkit/crashreporter/test/unit_ipc/test_content_annotation.js | test failed (with xpcshell return code: 0), see following log: >>>>>>> TEST-INFO | (xpcshell/head.js) | test 1 pending parent: TEST-INFO | (xpcshell/head.js) | test 2 pending parent: TEST-INFO | (xpcshell/head.js) | test 2 finished parent: TEST-INFO | (xpcshell/head.js) | running event loop [Child 3010] ###!!! ABORT: Intentional crash: file ../../../../../toolkit/crashreporter/test/nsTestCrasher.cpp, line 62 [Child 3010] ###!!! ABORT: Intentional crash: file ../../../../../toolkit/crashreporter/test/nsTestCrasher.cpp, line 62 parent: TEST-INFO | (xpcshell/head.js) | test 2 pending parent: TEST-UNEXPECTED-FAIL | ../unit/head_crashreporter.js | false == true - See following stack: parent: JS frame :: /Users/cltbld/talos-slave/test/build/xpcshell/head.js :: do_throw :: line 440 parent: JS frame :: /Users/cltbld/talos-slave/test/build/xpcshell/head.js :: _do_check_eq :: line 534 parent: JS frame :: /Users/cltbld/talos-slave/test/build/xpcshell/head.js :: do_check_eq :: line 555 parent: JS frame :: /Users/cltbld/talos-slave/test/build/xpcshell/head.js :: do_check_true :: line 569 parent: JS frame :: ../unit/head_crashreporter.js :: handleMinidump :: line 97 parent: JS frame :: ../unit/head_crashreporter.js :: <TOP_LEVEL> :: line 131 parent: JS frame :: /Users/cltbld/talos-slave/test/build/xpcshell/head.js :: <TOP_LEVEL> :: line 407 parent: TEST-INFO | (xpcshell/head.js) | exiting test parent: TEST-UNEXPECTED-FAIL | ../unit/head_crashreporter.js | Unexpected exception 2147500036, see following stack: undefined parent: TEST-INFO | (xpcshell/head.js) | exiting test parent: TEST-INFO | (xpcshell/head.js) | test 2 finished <<<<<<< Debug: TEST-INFO | /Users/cltbld/talos-slave/test/build/xpcshell/tests/toolkit/crashreporter/test/unit_ipc/test_content_annotation.js | running test ... command timed out: 1200 seconds without output, attempting to kill
Mark and Ted have theories about this in bug 706751.
This is probably the same root cause as that other bug. We're probably just failing at writing minidumps for a child process. philor: We could add OS version info to the xpcshell manifests, I'm not sure about how we'd do it, but the simple thing would just be something like: fails-if = os == "mac" && os_ver == "10.7"
This is holding xpcshell hostage on both opt and debug on both 10.7 and 10.8 now. How about if we disable this test on all-mac, and then pretend that someone will someday work on disabling it only on 10.7, 10.8, 10.9, 10.10 and 11.0?
Blocks: 786084
Summary: unit_ipc/test_content_annotation.js fails on OS X 10.7 → unit_ipc/test_content_annotation.js fails on OS X 10.7 and 10.8
I think it'd be good if we could get someone just to debug the test on 10.7 ;-) I'm convinced it probably wouldn't take long to do with access to full debug etc. I'm just not in a position to upgrade atm. On debug this is a timeout, on opt it fails. So I think this is a duplicate of bug 706751.
Attached patch disableSplinter Review
That sounds good to me: bug 706751 can be about the test failing some other bug someone can file if they want to fix it can be about reenabling it on 10.6 instead of fixing the test failure on 10.7 and 10.8 this bug can be about disabling it on OS X before we reach the first anniversary of running the other one thousand four hundred and ninety two xpcshell tests hidden on 10.7
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Comment on attachment 657117 [details] [diff] [review] disable Landed in https://hg.mozilla.org/integration/mozilla-inbound/rev/ca6ad4ab3366, backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/ab525ba5eb19 - it looks rather like if you produce a net-empty xpcshell.ini, you break the build. This has not, as it turns out, done much to ameliorate my hatred for this test.
Attachment #657117 - Attachment is obsolete: true
Comment on attachment 657117 [details] [diff] [review] disable Yep, having a pretty nice evening. Apparently that was not my bustage, just a fine coincidental meeting with a fresh way for the Mac buildslaves to fail. Relanded in https://hg.mozilla.org/integration/mozilla-inbound/rev/5fd0de490937, and opt and debug xpcshell on 10.7 and 10.8 are unhidden there.
Attachment #657117 - Attachment is obsolete: false
Attachment #657117 - Flags: checkin+
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: