Closed Bug 1477202 Opened 6 years ago Closed 6 years ago

Don't ship PerTestCoverageUtils.jsm to users

Categories

(Testing :: Code Coverage, enhancement)

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

Tracking

(firefox63 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla63
Tracking Flags:
Tracking Status
firefox63 --- fixed

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file)

Patch
3.52 KB, patch
florian
: review+
Details | Diff | Splinter Review
Because of bug 1344267, we'd ship PerTestCoverageUtils.jsm to users, but we don't need to.
Summary: Don't ship PerTestCoverageUtils.jsm in → Don't ship PerTestCoverageUtils.jsm to users
Attached patch Patch — — Splinter Review
Attachment #8993617 - Flags: review?(florian)
Comment on attachment 8993617 [details] [diff] [review] Patch Review of attachment 8993617 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me, assuming you have verified that the MOZ_CODE_COVERAGE ifdefs work and that you don't need to add anything in 'DEFINES' in moz.build.
Attachment #8993617 - Flags: review?(florian) → review+
Pushed by mcastelluccio@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/75aa72e75bee Avoid shipping PerTestCoverageUtils.jsm to users by only adding it to Marionette when MOZ_CODE_COVERAGE is defined. r=florian
https://hg.mozilla.org/mozilla-central/rev/75aa72e75bee
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: