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

RESOLVED WONTFIX

Status

Testing
General
RESOLVED WONTFIX
5 years ago
11 days ago

People

(Reporter: emorley, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
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".
(Reporter)

Updated

5 years ago
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
Last Resolved: 11 days ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.