Closed Bug 1316738 Opened 6 years ago Closed 6 years ago

Remove XPCOM test TestInterfaceInfo

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla52
Tracking Flags:
Tracking Status
firefox52 --- fixed

People

(Reporter: erahm, Assigned: erahm)

References

Details

Attachments

(1 file)

Remove XPCOM test TestInterfaceInfo
5.63 KB, patch
froydnj
: review+
Details | Diff | Splinter Review 
This removes a GeckoSimpleProgram that is neither built nor run.

MozReview-Commit-ID: 5ERreh9Gksw
I think we could get away with leaving this around since it's not built, but it seems cleaner to get rid of it. Converting to a gtest would also present some trickiness as it depends on command line arguments.
Attachment #8809607 - Flags: review?(nfroyd)
Comment on attachment 8809607 [details] [diff] [review]
Remove XPCOM test TestInterfaceInfo

Review of attachment 8809607 [details] [diff] [review]:
-----------------------------------------------------------------

Seems useful, but trying to remove the dependency on command-line arguments probably isn't worth it.
Attachment #8809607 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/47c167305517
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.