Closed Bug 1576697 Opened 5 years ago Closed 5 years ago

Introduce LauncherResult in WinHeaderOnlyUtils.h instead of WindowsErrorResult

Categories

(Firefox :: Launcher Process, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 71
Tracking Status
firefox71 --- fixed

People

(Reporter: toshi, Assigned: toshi)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Currently the functions in WinHeaderOnlyUtils.h use WindowsErrorResult to handle an error. For fine-grained error analysis, we can use LauncherResult because it saves a filename and a line number.

It will be useful especially to analyze the launcher result failures.

Component: Widget: Win32 → Launcher Process
Product: Core → Firefox

WindowsErrorResult is a class to hold either a value or a Windows error
code based on the Result template. We also have LauncherResult for the
same purpose, which was introduced as a part of the launcher process feature
afterward. The difference is LauncherResult holds a filename and line
number along with an error code.

This patch integrates LauncherResult.h into WinHeaderOnlyUtils.h so that we
can use LauncherResult more broadly.

Pushed by aklotz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7a3378adb340
Use LauncherResult in WinHeaderOnlyUtils.h instead of WindowsErrorResult. r=aklotz
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Blocks: 1530541
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: