minor #37255 [AssetBundle] Move xml service configuration to php (NguyenTruongLinh)

This PR was squashed before being merged into the 5.2-dev branch.

Discussion
----------

[AssetBundle] Move xml service configuration to php

| Q             | A
| ------------- | ---
| Branch?       | master for features / 3.4, 4.4, 5.0 or 5.1 for bug fixes <!-- see below -->
| Bug fix?      | yes/no
| New feature?  | yes/no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | yes/no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | Fix #... <!-- prefix each issue number with "Fix #", if any -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->
<!--
Replace this notice by a short README for your feature/bugfix. This will help people
understand your PR and can be used as a start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - Never break backward compatibility (see https://symfony.com/bc).
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too.)
 - Features and deprecations must be submitted against branch master.
-->

Commits
-------

74650610ff [AssetBundle] Move xml service configuration to php

Author: Tobion

DependencyInjection/FrameworkExtension.php

@@ -315,7 +315,7 @@ public function load(array $configs, ContainerBuilder $container)
                 throw new LogicException('Asset support cannot be enabled as the Asset component is not installed. Try running "composer require symfony/asset".');
             }
 
-            $this->registerAssetsConfiguration($config['assets'], $container, $loader);
+            $this->registerAssetsConfiguration($config['assets'], $container, $phpLoader);
         }
 
         if ($this->messengerConfigEnabled = $this->isConfigEnabled($container, $config['messenger'])) {
@@ -990,9 +990,9 @@ private function registerRequestConfiguration(array $config, ContainerBuilder $c
         }
     }
 
-    private function registerAssetsConfiguration(array $config, ContainerBuilder $container, XmlFileLoader $loader)
+    private function registerAssetsConfiguration(array $config, ContainerBuilder $container, PhpFileLoader $loader)
     {
-        $loader->load('assets.xml');
+        $loader->load('assets.php');
 
         if ($config['version_strategy']) {
             $defaultVersion = new Reference($config['version_strategy']);

Resources/config/assets.php

@@ -0,0 +1,89 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+use Symfony\Component\Asset\Context\RequestStackContext;
+use Symfony\Component\Asset\Package;
+use Symfony\Component\Asset\Packages;
+use Symfony\Component\Asset\PathPackage;
+use Symfony\Component\Asset\UrlPackage;
+use Symfony\Component\Asset\VersionStrategy\EmptyVersionStrategy;
+use Symfony\Component\Asset\VersionStrategy\JsonManifestVersionStrategy;
+use Symfony\Component\Asset\VersionStrategy\RemoteJsonManifestVersionStrategy;
+use Symfony\Component\Asset\VersionStrategy\StaticVersionStrategy;
+
+return static function (ContainerConfigurator $container) {
+    $container->parameters()
+        ->set('asset.request_context.base_path', null)
+        ->set('asset.request_context.secure', null)
+    ;
+
+    $container->services()
+        ->set('assets.packages', Packages::class)
+            ->args([
+                service('assets.empty_package'),
+                [],
+            ])
+
+        ->alias(Packages::class, 'assets.packages')
+
+        ->set('assets.empty_package', Package::class)
+            ->args([
+                service('assets.empty_version_strategy'),
+            ])
+
+        ->set('assets.context', RequestStackContext::class)
+            ->args([
+                service('request_stack'),
+                param('asset.request_context.base_path'),
+                param('asset.request_context.secure'),
+            ])
+
+        ->set('assets.path_package', PathPackage::class)
+            ->abstract()
+            ->args([
+                abstract_arg('base path'),
+                abstract_arg('version strategy'),
+                service('assets.context'),
+            ])
+
+        ->set('assets.url_package', UrlPackage::class)
+            ->abstract()
+            ->args([
+                abstract_arg('base URLs'),
+                abstract_arg('version strategy'),
+                service('assets.context'),
+            ])
+
+        ->set('assets.static_version_strategy', StaticVersionStrategy::class)
+            ->abstract()
+            ->args([
+                abstract_arg('version'),
+                abstract_arg('format'),
+            ])
+
+        ->set('assets.empty_version_strategy', EmptyVersionStrategy::class)
+
+        ->set('assets.json_manifest_version_strategy', JsonManifestVersionStrategy::class)
+            ->abstract()
+            ->args([
+                abstract_arg('manifest path'),
+            ])
+
+        ->set('assets.remote_json_manifest_version_strategy', RemoteJsonManifestVersionStrategy::class)
+            ->abstract()
+            ->args([
+                abstract_arg('manifest url'),
+                service('http_client'),
+            ])
+    ;
+};