Closed Bug 827323 Opened 8 years ago Closed 3 years ago

AssertAppProcessPermission() does not output a TBPL-parsable error message when killing content processes

Categories

(Testing :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: emorley, Unassigned)

References

(Blocks 1 open bug)

Details

The current log output looks like:
{
41845 INFO TEST-PASS | /tests/content/base/test/test_messagemanager_assertpermission.html | Frame mm has assigned permission.
Security problem: Content process does not have `frobnaz' permission.  It will be killed.
process killed by signal 15
program finished with exit code -1
elapsedTime=396.386989
TinderboxPrint: mochitest-plain-1<br/><em class="testfail">T-FAIL</em>
Unknown Error: command finished with exit code: -1
========= Finished 'python mochitest/runtests.py ...' warnings (results: 1, elapsed: 6 mins, 38 secs) (at 2013-01-07 04:34:23.421614) =========
}

We should either:
a) Modify http://mxr.mozilla.org/mozilla-central/source/dom/ipc/AppProcessPermissions.cpp#43 to output a string that matches the normal assertion/ABORT formats (or something else that TBPL already matches).
b) Catch this in automation.py (/wherever) and output a TEST-UNEXPECTED-FAIL or similar.

If we do (a), we should still make sure that automation.py makes use of testLastSeen, so that we output something that includes the test name, and not just "process killed by signal 15".
Blocks: 820353
I like option (a).

This would be a great between-blocker bug to grab.
Mass closing bugs with no activity in 2+ years. If this bug is important to you, please re-open.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.