1335133 - Remove 'azureQuartz' reftest variable and all its usages

Status: RESOLVED FIXED

(Testing :: Reftest, defect)

Description

[Daniel Holbert [:dholbert]]

lsaltzman and mstange tell me that the 'azureQuartz' reftest.list variable can be assumed to always be false these days. (We use Skia as the backend on Mac now, as noted in bug 932958)

We should get rid of this reftest variable and all of its usages. Aryx, you've been heroically cleaning up obsolete cruft in reftest.list files recently - perhaps you'd be interested?

Comment 1

[Daniel Holbert [:dholbert]]

[This depends on bug 1334898 since there's some overlap in the affected lines, so we should base any changes here on top of bug 1334898.]

Comment 2

[Markus Stange [:mstange]]

The CoreGraphics Moz2D backend has been removed in bug 1258751, which rode the trains with Firefox 52.

Comment 3

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Attachment: Bug 1335133 - Remove 'azureQuartz' reftest variable and all its usages.

Review commit: https://reviewboard.mozilla.org/r/108380/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/108380/

Comment 4

[Daniel Holbert [:dholbert]]

Comment on attachment 8831885 [details]
Bug 1335133 - Remove 'azureQuartz' reftest variable and all its usages.

https://reviewboard.mozilla.org/r/108380/#review109530

This looks great - thanks!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c659e39a9861
Remove 'azureQuartz' reftest variable and all its usages. r=dholbert

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c659e39a9861