Closed Bug 1334898 Opened 8 years ago Closed 8 years ago

Remove annotations for Windows XP & Vista and OSX<10.9 from layout reftests

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox51 --- wontfix
firefox52 --- wontfix
firefox53 --- wontfix
firefox54 --- fixed

People

(Reporter: aryx, Assigned: aryx)

References

Details

Attachments

(2 files)

Because Gecko dropped support for these platforms.
(In reply to Sebastian Hengst [:aryx][:archaeopteryx] (needinfo on intermittent or backout) from comment #0) > Because Gecko dropped support for these platforms. Thanks for doing this cleanup! One request: for historical purposes [and to satisfy my due diligence as a reviewer], it'd be great if you could include links to the relevant announcement/newsgroup-thread/etc when filing bugs like this. (I was going to request that via needinfo, but it's easy enough for me to find & post here, so I'll just do that. :)) OS X pre-10.9 unsupport was announced here (effective version 49): https://blog.mozilla.org/futurereleases/2016/04/29/update-on-firefox-support-for-os-x/ WinXP/Vista unsupport was announced here, albeit with no mention of version numbers: https://blog.mozilla.org/futurereleases/2016/12/23/firefox-support-for-xp-and-vista/ I believe this newsgroup thread indicates that ~52 is the last supported version: https://groups.google.com/forum/?fromgroups#!msg/mozilla.dev.platform/W6pUpP_FbZY/-AEWVIemAgAJ;context-place=msg/mozilla.dev.platform/d5iOzXhMlm8/gr2esYAcAgAJ ...though I haven't recently read through the whole thread. But it looks like we should be safe assuming that we're no longer supporting these platforms in trunk (version 54).
(bug 1315083 comment 0 includes some WinXP context, too, further confirming that this is fine)
Depends on: 1315083
Comment on attachment 8831535 [details] Bug 1334898 - Remove annotations for Windows XP from layout reftests. https://reviewboard.mozilla.org/r/108082/#review109344 This patch looks good to me. I went through and WONTFIX'ed all the bugs that this patch is removing references to, BTW.
Attachment #8831535 - Flags: review?(dholbert) → review+
Comment on attachment 8831536 [details] Bug 1334898 - Remove annotations for OSX<10.9 from layout reftests. https://reviewboard.mozilla.org/r/108084/#review109352 r=me, with just one simplification that you should make here: ::: layout/reftests/canvas/reftest.list:63 (Diff revision 1) > ## Currently fails most places due to partial backout of bug 808288, see bug 837461. > ## (Marking "random" rather than "fails" because it would pass for people > ## if they have an Emoji font installed when running the tests.) > -## WAS: random-if(OSX==1007) != text-emoji.html text-emoji-notref.html > # With Skia canvas on OS X (bug 932958) it fails even on 10.8 and 10.10. > -random-if(cocoaWidget&&azureSkia) random-if(!cocoaWidget||OSX==1006||OSX==1007) != text-emoji.html text-emoji-notref.html > +random-if(cocoaWidget&&azureSkia) random-if(!cocoaWidget) != text-emoji.html text-emoji-notref.html You should just collapse away all the conditions here and just label this as "random". The random-if conditions cover all possible worlds and are needlessly complex historical cruft from a time when they didn't cover all possible worlds. Specifically: * cocoaWidget implies azureSkia these days, so the first condition here is really just "random-if(cocoaWidget)". * And when paired with the second condition ("random-if(!cocoaWidget)"), that covers all possible worlds. * So, this reftest is just "random". (Too bad.)
Attachment #8831536 - Flags: review?(dholbert) → review+
(I WONTFIX'ed a few old-Mac-OSX bugs whose annotations you're removing in part 2, too.)
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/mozilla-inbound/rev/64f855d83513 Remove annotations for Windows XP from layout reftests. r=dholbert https://hg.mozilla.org/integration/mozilla-inbound/rev/3b7d96464366 Remove annotations for OSX<10.9 from layout reftests. r=dholbert
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: