Replace Error.Code() with Error.ErrorName() for the debug message

RESOLVED FIXED in Firefox 58

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: alwu, Assigned: alwu)

Tracking

Other Branch
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

Attachments

(2 attachments)

The follow up bug from bug 1402206.

Use ErrorName() could better know the error detailed.
Comment on attachment 8913074 [details]
Bug 1403832 - part1 : replace Error.Code() with Error.ErrorName().

https://reviewboard.mozilla.org/r/184482/#review189794

::: commit-message-70158:3
(Diff revision 1)
> +Bug 1403832 - part1 : replace Error.Code() with Error.ErrorName().
> +
> +Use ErrorName() could better know the error detailed.

"Use ErrorName() as it provides more useful information for the error detail"
Attachment #8913074 - Flags: review?(jyavenard) → review+
Comment on attachment 8913075 [details]
Bug 1403832 - part2 : reuse ErrorName() in MediaResult::Description().

https://reviewboard.mozilla.org/r/184484/#review189796
Attachment #8913075 - Flags: review?(jyavenard) → review+
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/89061adfacb3
part1 : replace Error.Code() with Error.ErrorName(). r=jya
https://hg.mozilla.org/integration/autoland/rev/30d8969e3ec6
part2 : reuse ErrorName() in MediaResult::Description(). r=jya
https://hg.mozilla.org/mozilla-central/rev/89061adfacb3
https://hg.mozilla.org/mozilla-central/rev/30d8969e3ec6
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.