Issue #3504559 by berdir: AssetResolver may load the same cache several time on a single page with bigpipe

Author: catch

core.services.yml

@@ -1710,9 +1710,21 @@ services:
     class: Drupal\Core\Asset\LibraryDependencyResolver
     arguments: ['@library.discovery']
   Drupal\Core\Asset\LibraryDependencyResolverInterface: '@library.dependency_resolver'
+  # Set up a cache chain for asset caching as the same assets may be
+  # requested several times, non-public as they are not meant to be reused.
+  cache.asset_memory:
+    class: Drupal\Core\Cache\MemoryCache\MemoryCache
+    arguments: ['@datetime.time']
+    public: false
+  cache.asset_chain:
+    class: Drupal\Core\Cache\BackendChain
+    public: false
+    calls:
+      - [ appendBackend, [ '@cache.asset_memory' ] ]
+      - [ appendBackend, [ '@cache.data' ] ]
   asset.resolver:
     class: Drupal\Core\Asset\AssetResolver
-    arguments: ['@library.discovery', '@library.dependency_resolver', '@module_handler', '@theme.manager', '@language_manager', '@cache.data', '@theme_handler']
+    arguments: ['@library.discovery', '@library.dependency_resolver', '@module_handler', '@theme.manager', '@language_manager', '@cache.asset_chain', '@theme_handler']
   Drupal\Core\Asset\AssetResolverInterface: '@asset.resolver'
   info_parser:
     class: Drupal\Core\Extension\InfoParser

modules/navigation/tests/src/FunctionalJavascript/PerformanceTest.php

@@ -73,10 +73,10 @@ public function testLogin(): void {
 
     $expected = [
       'QueryCount' => 4,
-      'CacheGetCount' => 59,
+      'CacheGetCount' => 58,
       'CacheGetCountByBin' => [
         'config' => 11,
-        'data' => 6,
+        'data' => 5,
         'discovery' => 10,
         'bootstrap' => 6,
         'dynamic_page_cache' => 2,

profiles/demo_umami/tests/src/FunctionalJavascript/OpenTelemetryAuthenticatedPerformanceTest.php

@@ -51,11 +51,11 @@ public function testFrontPageAuthenticatedWarmCache(): void {
 
     $expected = [
       'QueryCount' => 4,
-      'CacheGetCount' => 42,
+      'CacheGetCount' => 41,
       'CacheGetCountByBin' => [
         'config' => 22,
         'discovery' => 5,
-        'data' => 7,
+        'data' => 6,
         'bootstrap' => 4,
         'dynamic_page_cache' => 2,
         'render' => 2,

profiles/standard/tests/src/FunctionalJavascript/StandardPerformanceTest.php

@@ -469,7 +469,7 @@ protected function testLoginBlock(): void {
     $this->assertSame($expected_queries, $recorded_queries);
     $expected = [
       'QueryCount' => 18,
-      'CacheGetCount' => 105,
+      'CacheGetCount' => 104,
       'CacheSetCount' => 1,
       'CacheDeleteCount' => 1,
       'CacheTagInvalidationCount' => 0,