remove DoFindInReadable from nsExceptionHandler

RESOLVED FIXED in Firefox 53

Status

()

Toolkit
Crash Reporting
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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.
(Assignee)

Comment 1

2 years ago
Created attachment 8812284 [details] [diff] [review]
remove DoFindInReadable from nsExceptionHandler
Attachment #8812284 - Flags: review?(ted)
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+

Comment 3

2 years ago
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/11552f8437ae
remove DoFindInReadable from nsExceptionHandler; r=ted.mielczarek

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/11552f8437ae
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.