1949652 - mozperftest testing/performance/perftest_youtube_link.js actually loads politico

Status: NEW

(Testing :: mozperftest, defect, P3)

Description

[Ted Campbell [:tcampbell]]

./mach perftest testing/performance/perftest_youtube_link.js goes to politco.com instead of youtube

Comment 1

[Kash Shampur [:kshampur] ⌚EST]

Hi Andrew!
looking at the blame for this https://searchfox.org/mozilla-central/rev/fcfb558f8946f3648d962576125af46bf6e2910a/testing/performance/perftest_youtube_link.js

leads to https://bugzilla.mozilla.org/show_bug.cgi?id=1627623

I know it has been a few years but would you have additional context on why politico was used here?

Comment 2

[Andrew Creskey [:acreskey]]

Ah, interesting find!

It looks like a bug to me -- since we already have a "politico.com" test.

https://searchfox.org/mozilla-central/rev/fcfb558f8946f3648d962576125af46bf6e2910a/testing/perfdocs/generated/mozperftest.rst#326-339

But I don't think any of these tests are actually used -- they are more examples that were made when building mozperftest.

Comment 3

[Ted Campbell [:tcampbell]]

Do you think we can delete some of these or at perhaps put in an examples directory? Our collection of perftests across the tree is pretty clutter and more and more of them seem broken on different platforms.