feature symfony#59544 [AssetMapper] Fix CssCompiler matches url in comments (smnandre)

This PR was squashed before being merged into the 6.4 branch.

Discussion
----------

[AssetMapper] Fix CssCompiler matches url in comments

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Issues        | Fix symfony#59512
| License       | MIT

Add a check to avoid matching url in comments

Commits
-------

9a43703 [AssetMapper] Fix CssCompiler matches url in comments

Author: fabpot

src/Symfony/Component/AssetMapper/Compiler/CssAssetUrlCompiler.php

@@ -35,7 +35,32 @@ public function __construct(
 
     public function compile(string $content, MappedAsset $asset, AssetMapperInterface $assetMapper): string
     {
-        return preg_replace_callback(self::ASSET_URL_PATTERN, function ($matches) use ($asset, $assetMapper) {
+        preg_match_all('/\/\*|\*\//', $content, $commentMatches, \PREG_OFFSET_CAPTURE);
+
+        $start = null;
+        $commentBlocks = [];
+        foreach ($commentMatches[0] as $match) {
+            if ('/*' === $match[0]) {
+                $start = $match[1];
+            } elseif ($start) {
+                $commentBlocks[] = [$start, $match[1]];
+                $start = null;
+            }
+        }
+
+        return preg_replace_callback(self::ASSET_URL_PATTERN, function ($matches) use ($asset, $assetMapper, $commentBlocks) {
+            $matchPos = $matches[0][1];
+
+            // Ignore matchs inside comments
+            foreach ($commentBlocks as $block) {
+                if ($matchPos > $block[0]) {
+                    if ($matchPos < $block[1]) {
+                        return $matches[0][0];
+                    }
+                    break;
+                }
+            }
+
             try {
                 $resolvedSourcePath = Path::join(\dirname($asset->sourcePath), $matches[1]);
             } catch (RuntimeException $e) {

src/Symfony/Component/AssetMapper/Tests/Compiler/CssAssetUrlCompilerTest.php

@@ -114,6 +114,36 @@ public static function provideCompileTests(): iterable
             'expectedOutput' => 'body { background: url("https://cdn.io/images/bar.png"); }',
             'expectedDependencies' => [],
         ];
+
+        yield 'ignore_comments' => [
+            'input' => 'body { background: url("images/foo.png"); /* background: url("images/bar.png"); */ }',
+            'expectedOutput' => 'body { background: url("images/foo.123456.png"); /* background: url("images/bar.png"); */ }',
+            'expectedDependencies' => ['images/foo.png'],
+        ];
+
+        yield 'ignore_comment_after_rule' => [
+            'input' => 'body { background: url("images/foo.png"); } /* url("images/need-ignore.png") */',
+            'expectedOutput' => 'body { background: url("images/foo.123456.png"); } /* url("images/need-ignore.png") */',
+            'expectedDependencies' => ['images/foo.png'],
+        ];
+
+        yield 'ignore_comment_within_rule' => [
+            'input' => 'body { background: url("images/foo.png") /* url("images/need-ignore.png") */; }',
+            'expectedOutput' => 'body { background: url("images/foo.123456.png") /* url("images/need-ignore.png") */; }',
+            'expectedDependencies' => ['images/foo.png'],
+        ];
+
+        yield 'ignore_multiline_comment_after_rule' => [
+            'input' => 'body {
+                background: url("images/foo.png"); /*
+                url("images/need-ignore.png") */
+            }',
+            'expectedOutput' => 'body {
+                background: url("images/foo.123456.png"); /*
+                url("images/need-ignore.png") */
+            }',
+            'expectedDependencies' => ['images/foo.png'],
+        ];
     }
 
     public function testCompileFindsRelativeFilesViaSourcePath()