Remove whitelist/blacklist from frontend performance tests
Categories
(Firefox :: General, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox79 | --- | fixed |
People
(Reporter: Gijs, Assigned: Gijs)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Let's get rid of this terminology.
Assignee | ||
Comment 1•7 months ago
|
||
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/9c35e1ec27d8 remove use of whitelist/blacklist from front-end performance tests, r=mconley
Assignee | ||
Comment 3•7 months ago
|
||
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?
Assignee | ||
Comment 4•7 months ago
|
||
(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•7 months ago
|
||
:Gijs thank you for checking this and removing the regressions.
I will further investigate.
Comment 6•7 months ago
|
||
bugherder |
Assignee | ||
Comment 7•7 months ago
|
||
(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. :-)
Description
•