Closed Bug 1362478 Opened 8 years ago Closed 8 years ago

Content processes can't update coverage counters due to sandboxing

Categories

(Testing :: Code Coverage, enhancement)

Product:
Testing
Component:
Code Coverage
Version:
Version 3
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(firefox55 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file)

Patch
1005 bytes, patch
jmaher
: review+
Details | Diff | Splinter Review
I've tested disabling the sandbox locally with the example from bug 1362052 and the coverage looks right to me.
Attached patch Patch — — Splinter Review
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #8864946 - Flags: review?(jmaher)
Comment on attachment 8864946 [details] [diff] [review] Patch Review of attachment 8864946 [details] [diff] [review]: ----------------------------------------------------------------- awesome find
Attachment #8864946 - Flags: review?(jmaher) → review+
Pushed by mcastelluccio@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/996124fe6f8d Disable sandbox in the code coverage build, as it prevents content processes from updating gcda files. r=jmaher
https://hg.mozilla.org/mozilla-central/rev/996124fe6f8d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: