Closed Bug 1331170 Opened 3 years ago Closed 3 years ago

Remove gfx/thebes checks for Windows Vista and 7+

Categories

(Core :: Graphics, defect, P3)

51 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox50 --- wontfix
firefox51 --- wontfix
firefox52 --- wontfix
firefox53 --- fixed

People

(Reporter: cpeterson, Assigned: cpeterson)

References

(Depends on 1 open bug)

Details

Attachments

(2 files)

Now that Firefox 53 no longer supports Windows XP or Vista, we can remove unnecessary Windows version checks for XP, Vista, and 7.
There is still one Vista check in gfx/thebes that I didn't remove (yet) because it unlocks a nest of ClearType code (bug 504698) that can be removed because it is only used on Windows XP.
Flipped review to jrmuizel (I think, can't really tell with Reviewboard). He might have an opinion on whether we should tear out this code right away.
Attachment #8826867 - Flags: review?(dvander) → review?(jmuizelaar)
Attachment #8826868 - Flags: review?(dvander) → review?(jmuizelaar)
Comment on attachment 8826867 [details]
Bug 1331170 - Part 1: Remove some gfx/thebes checks for Windows Vista+.

https://reviewboard.mozilla.org/r/104718/#review106090
Attachment #8826867 - Flags: review?(jmuizelaar) → review+
Comment on attachment 8826868 [details]
Bug 1331170 - Part 2: Remove gfx/thebes checks for Windows 7+.

https://reviewboard.mozilla.org/r/104720/#review106092
Attachment #8826868 - Flags: review?(jmuizelaar) → review+
Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/32e8088d94d8
Part 1: Remove some gfx/thebes checks for Windows Vista+. r=jrmuizel
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9fe1bab1024
Part 2: Remove gfx/thebes checks for Windows 7+. r=jrmuizel
https://hg.mozilla.org/mozilla-central/rev/32e8088d94d8
https://hg.mozilla.org/mozilla-central/rev/a9fe1bab1024
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.