Remove performance audits that were replaced with insight audits

[connorjclark]

ref #16584

critical-request-chains
dom-size
duplicated-javascript
efficient-animated-content
font-display
largest-contentful-paint-element
layout-shifts
lcp-lazy-loaded
modern-image-formats
non-composited-animations
prioritize-lcp-image
redirects
render-blocking-resources
server-response-time
third-party-summary
unsized-images
uses-http2
uses-long-cache-ttl
uses-optimized-images
uses-rel-preconnect
uses-responsive-images
uses-responsive-images-snapshot
uses-text-compression
viewport
work-during-interaction

[connorjclark]

Slight change in plans.

Will keep a few of these audits, but make them hidden in the report:

• layout-shifts
• server-response-time
• redirects

Reasoning: it's simple to keep these and it'll be nice to not break this interface for programmatic users.

Will keep two audits (and continue showing them in report):

• non-composited-animations
• unsized-images

Reasoning:

b/448406176 🔒

Non-composited animations are one of the CLS culprits we have, and so we were planning to remove the Lighthouse audit for it. However, that overlooked that the CLS culprits insight will only show a bad animation if a shift was detected. There can still be bad animations with no shift.

So we should have an insight that just surfaces all the non-composited animations, like the Lighthouse audit does. We show warnings for bad animations in the animations track, so it makes sense to have an insight for it too.

In the meantime, let's not remove the Lighthouse audit.

Ditto for unsized images.

[benschwarz]

Thanks for posting the whole list @connorjclark, appreciated.

[connorjclark]

I updated my second comment to also include unsized-images – we also want to make a separate insight for that now, so until then the audit will remain.