Optimize AssetResolver caching

Anonymous performance tests show the improvement in cache hit rate - one less cache set.

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Not entirely sure how to best test this

So going to lean on PerformanceTest change that show the count go down

All threads appear to be answered.

Tests are all green

So going to go out on a limb and mark it, apologize if there is a better way to test.

📌 Return early in AssetResolver::getJsAssets() and AssetResolver::getCssAssets() when there are no libraries to load Needs review was manually testable because it changed the query string on asset aggregate URLs, but this one isn't really - purely a caching optimization internal to the class.

Committed and pushed 181827afe6 to 11.x and 7cfb8bbd48 to 10.3.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.