remove DoFindInReadable from nsExceptionHandler

RESOLVED FIXED in Firefox 53

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

We now have an overload for the simple case of finding |key| in some
|string|, so we don't need this utility function anymore, which
presumably dates from when said overload did not exist or was
inaccessible due to library visibility, or something.
Comment on attachment 8812284 [details] [diff] [review]
remove DoFindInReadable from nsExceptionHandler

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

I think it didn't exist and I was cursing our string classes when I wrote that. Thanks!
Attachment #8812284 - Flags: review?(ted) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/11552f8437ae
remove DoFindInReadable from nsExceptionHandler; r=ted.mielczarek
https://hg.mozilla.org/mozilla-central/rev/11552f8437ae
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.