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

Status: RESOLVED WONTFIX

(Testing :: General, defect)

Description

[Ed Morley [:emorley]]

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".

Comment 1

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

I like option (a).

This would be a great between-blocker bug to grab.

Comment 2

[Geoff Brown [:gbrown]]

Mass closing bugs with no activity in 2+ years. If this bug is important to you, please re-open.