enable test_fontVariationsAPI.xul for windows10 now that we run on v1803

RESOLVED FIXED in Firefox 67

Status

enhancement
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: jmaher, Assigned: jmaher)

Tracking

(Blocks 1 bug)

Version 3
mozilla67
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(1 attachment)

enable test_fontVariationsAPI.xul for windows10 now that we run on v1803

Comment 2

4 months ago
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2d5fbe3bb7f6
enable test_fontVariationsAPI.xul for windows10 now that we run on v1803. r=jfkthame

Comment 3

4 months ago
bugherder
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Assignee: nobody → jmaher

Any reason we did this with Win64 instead of using os_version?

Flags: needinfo?(jmaher)

my understanding is 32 bit will not have support and we only have one version of windows10-64. short answer is there is some laziness mixed with the reality that this only runs in a very limited set of configs. Would you like me to change this? (fyi, I just enabled this for linux64 about 10 minutes ago)

Flags: needinfo?(jmaher)

AFAIK, 32 vs 64 was mainly due to what we happen to run in CI vs. anything on Microsoft's end. I think os_version is a bit better for the off chance of anybody running tests locally (i.e. Win7 64-bit would fail and cause confusion). I believe we'd have to use a version 10.0, but I think it's OK since something like 90% of our users are on a version which supports this OS feature.

You need to log in before you can comment on or make changes to this bug.