Closed Bug 1041571 Opened 6 years ago Closed 5 years ago

Add some basic leak testing for nsSystemStatusBarCocoa

Categories

(Core :: Widget: Cocoa, defect)

All
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: mstange, Assigned: mstange)

References

Details

Attachments

(1 file)

There's currently no test coverage of nsSystemStatusBarCocoa, so we don't even benefit from leak checking (which is done automatically in test runs with debug builds). Adding some basic test coverage is simple enough that there's really no excuse not to do it.
Attached patch v1Splinter Review
This test can be run locally using
mach mochitest widget/tests/test_system_status_bar.xul
Attachment #8459630 - Flags: review?(smichaud)
Comment on attachment 8459630 [details] [diff] [review]
v1

Looks fine to me.
Attachment #8459630 - Flags: review?(smichaud) → review+
https://hg.mozilla.org/mozilla-central/rev/cf6966a86753
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.