Remove XPCOM test TestStringAPI

RESOLVED FIXED in Firefox 52

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: erahm, Assigned: erahm)

Tracking

unspecified
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

Convert xpcom/tests/TestStringAPI.cpp to a gtest and move to xpcom/tests/gtest/.
Once the xpcom glue library is removed we will no longer need an external
StringAPI, so this test can go away.

MozReview-Commit-ID: DdGV2dWvlK1
Attachment #8806841 - Flags: review?(nfroyd)
Assignee: nobody → erahm
Status: NEW → ASSIGNED
This one's a bit of a catch-22. It would be nice to remove this in the same patch that kills the xpcom glue lib, but we can't kill the glue lib until all GeckoCppUnitTests are gone.
Summary: Convert XPCOM test TestStringAPI to a gtest → Remove XPCOM test TestStringAPI
Comment on attachment 8806841 [details] [diff] [review]
Remove test TestStringAPI

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

\o/
Attachment #8806841 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/51d34a672785
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.