1647787 - Remove whitelist/blacklist from frontend performance tests

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[:Gijs (he/him)]

Let's get rid of this terminology.

Comment 1

[:Gijs (he/him)]

Attachment: Bug 1647787 - remove use of whitelist/blacklist from front-end performance tests, r?mconley

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9c35e1ec27d8
remove use of whitelist/blacklist from front-end performance tests, r=mconley

Comment 3

[:Gijs (he/him)]

This patch just changed comments in the urlbar test, nothing else, so it can't have caused those regressions. I also expect it won't have caused the layout.dpi regression, and instead the cause is whatever caused the urlbar tests to start failing. Please can you doublecheck the regression range?

Comment 4

[:Gijs (he/him)]

(the prefs "regression" was just a change in the message, but the intermittent was already known. I've updated the summary of the older bug.)

Comment 5

[Natalia Csoregi [:nataliaCs]]

:Gijs thank you for checking this and removing the regressions.
I will further investigate.

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/9c35e1ec27d8

Comment 7

[:Gijs (he/him)]

(In reply to Natalia Csoregi [:nataliaCs] from comment #5)

:Gijs thank you for checking this and removing the regressions.
I will further investigate.

I'll add that I only just realized that the 2 urlbar-test-related bugs are also TV bugs; because it's a TV failure, it's unclear when that would have been introduced (as it presents when repeatedly running the same test), although it's possible there are similar/identical non-TV intermittents already on file, that are just showing up as TV issues now that the test is being changed.

Still, changing a bit of English text in a comment in the test of course shouldn't be influencing how the test works. :-)