Closed Bug 1631441 Opened 3 years ago Closed 4 months ago

Remove XBL tests

Categories

(Core :: XBL, task)

Product:
Core
Component:
XBL
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
109 Branch
Tracking Flags:
Tracking Status
firefox109 --- fixed

People

(Reporter: ntim, Assigned: marco)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1631441 - Remove no longer relevant tests with XBL bindings. r=#layout-reviewers,#firefox-style-system-reviewers,#firefox-svg-reviewers
48 bytes, text/x-phabricator-request
Details | Review

Is there any reason for keeping these tests? Does removing them speed up any aspect of the program? How much of a patch would it be to remove?

Brian, can we remove these tests? It would unlock finding more uncovered files that we could get rid of.

Flags: needinfo?(bgrinstead)

(In reply to Marco Castelluccio [:marco] from comment #2)

Brian, can we remove these tests? It would unlock finding more uncovered files that we could get rid of.

I don't see any reason not to remove them, so long as they are/were only covering XBL functionality and not just accessing moz-bindings as part of a broader test. Though I'd be surprised if these are uniquely providing coverage for other files.

Flags: needinfo?(bgrinstead)
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dab79b4f8a22
Remove no longer relevant tests with XBL bindings. r=emilio DONTBUILD

https://hg.mozilla.org/mozilla-central/rev/dab79b4f8a22

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
status-firefox109: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
You need to log in before you can comment on or make changes to this bug.