perf(@angular-devkit/build-angular): avoid extra string creation with…

… no sourcemaps for esbuild sass

When using the experimental esbuild-based browser application builder with Sass stylesheets, an
additional string creation for each output stylesheet will now be avoided when sourcemaps are disabled.

(cherry picked from commit a910ae9)

packages/angular_devkit/build_angular/src/builders/browser-esbuild/sass-plugin.ts

@@ -42,7 +42,7 @@ export function createSassPlugin(options: { sourcemap: boolean; loadPaths?: stri
 
           return {
             loader: 'css',
-            contents: `${css}\n${sourceMapToUrlComment(sourceMap)}`,
+            contents: sourceMap ? `${css}\n${sourceMapToUrlComment(sourceMap)}` : css,
             watchFiles: loadedUrls.map((url) => fileURLToPath(url)),
             warnings,
           };
@@ -68,11 +68,7 @@ export function createSassPlugin(options: { sourcemap: boolean; loadPaths?: stri
   };
 }
 
-function sourceMapToUrlComment(sourceMap: CompileResult['sourceMap']): string {
-  if (!sourceMap) {
-    return '';
-  }
-
+function sourceMapToUrlComment(sourceMap: Exclude<CompileResult['sourceMap'], undefined>): string {
   const urlSourceMap = Buffer.from(JSON.stringify(sourceMap), 'utf-8').toString('base64');
 
   return `/*# sourceMappingURL=data:application/json;charset=utf-8;base64,${urlSourceMap} */`;