1498213 - Expose Windows build number Windows in AppConstants or sysinfo

Status: ASSIGNED

(Toolkit :: General, defect, P5)

Description

[Dão Gottwald [::dao]]

In bug 1496322, I needed access to the Windows 10 build number, and for lack of a better alternative, I used TelemetryEnvironment for that, which is somewhat wrong since my use case has nothing to with Telementry, plus the API is async, and as Gijs noted, "Having to wait for TelemetryEnvironment.onInitialized() is a bit unfortunate, esp. given that AFAICT getting the actual version number info is sync."

Comment 1

[Mike Conley (:mconley) (:⚙️) (Catching up on needinfos)]

Attachment: Bug 1498213 - Expose more version information for Windows on nsISystemInfo.osVersionInfo. r?Gijs

Comment 2

[Mike Conley (:mconley) (:⚙️) (Catching up on needinfos)]

Attachment: Bug 1498213 - Use nsISystemInfo.osVersionInfo for TelemetryEnvironment and in browser.js. r?Gijs

TelemetryEnvironment was using jsctypes to get the version information,
which we can now skip. browser.js was using TelemetryEnvironment to determine
if the Windows 10 build was greater or equal to the 1809 release. Both now
can use nsISystemInfo.osVersionInfo directly.

Depends on D63373