1904949 - Add `cause` codes to exceptions that we expect to map to errors shown to the user

Status: RESOLVED FIXED

(Firefox :: Profile Backup, task, P3)

Description

[Mike Conley (:mconley) (:⚙️)]

We have a whole bunch of exceptions that can get thrown in the BackupService and related components. Some of those result in errors that need to be shown to users.

What I think we should do is to have a cause get supplied for every error we expect to be shown to users. The value of the cause should be a numeric constant defined in ArchiveUtils.

Then, if and when we need to show an exception to a user, we can hopefully map from the cause constant to the appropriate localized string for the context.

Filed as a meta, because I figure we'll probably do this piecemeal.

Comment 1

[Stephen Thompson]

Attachment: Bug 1904949 - Add cause error codes to backup service r=#backup-reviewers

This introduces error code causes that can be used by UI components to present context-relevant error text. This starts throwing errors with cause error codes in parts of the backup codebase, but there are still areas (especially at the boundary with web workers) where cause codes need to be introduced and handled.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a48d151dddff
Add cause error codes to backup service r=backup-reviewers,mconley

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/a48d151dddff