Open Bug 2015092 Opened 3 days ago Updated 7 hours ago

Refactor error handling in ConsoleClient

Categories

(Firefox Enterprise :: Client, enhancement, P2)

Product:
Firefox Enterprise
Component:
Client
Platform:
x86_64
Linux
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: gerard-majax, Unassigned)

References

(Depends on 1 open bug)

Details

As outlined in https://github.com/mozilla/enterprise-firefox/pull/358#discussion_r2774445599 the mapping done for handling error reporting has a few limitations because we are re-doing what exists in nsDocShell::DisplayLoadError. Especially when bug 1790187 gets fixed, we need to revisit this.

Quoting from the PR:

The error code to string id mapping ultimately doesn't belong in the ConsoleClient. I'd rather have it only return the status and let window.js do the error name to string id mapping in one place. I think the current approach is not a good solution. But given the urgency of this, this is not a blocker.

Going to lower priority since there's no pressing customer ask here and it depends on Firefox team changes.

Severity: S2 → S3
You need to log in before you can comment on or make changes to this bug.