Don't enable tracing unless the profiler is enabled

Author: nicolas-grekas

DependencyInjection/FrameworkExtension.php

@@ -106,6 +106,7 @@
 use Symfony\Component\HttpKernel\DataCollector\DataCollectorInterface;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;
 use Symfony\Component\HttpKernel\Log\DebugLoggerConfigurator;
+use Symfony\Component\HttpKernel\Profiler\ProfilerStateChecker;
 use Symfony\Component\JsonStreamer\Attribute\JsonStreamable;
 use Symfony\Component\JsonStreamer\JsonStreamWriter;
 use Symfony\Component\JsonStreamer\StreamReaderInterface;
@@ -963,6 +964,11 @@ private function registerProfilerConfiguration(array $config, ContainerBuilder $
         $loader->load('collectors.php');
         $loader->load('cache_debug.php');
 
+        if (!class_exists(ProfilerStateChecker::class)) {
+            $container->removeDefinition('profiler.state_checker');
+            $container->removeDefinition('profiler.is_disabled_state_checker');
+        }
+
         if ($this->isInitializedConfigEnabled('form')) {
             $loader->load('form_debug.php');
         }

Resources/config/debug.php

@@ -25,6 +25,7 @@
                 service('debug.stopwatch'),
                 service('logger')->nullOnInvalid(),
                 service('.virtual_request_stack')->nullOnInvalid(),
+                service('profiler.is_disabled_state_checker')->nullOnInvalid(),
             ])
             ->tag('monolog.logger', ['channel' => 'event'])
             ->tag('kernel.reset', ['method' => 'reset'])

Resources/config/profiling.php

@@ -16,6 +16,7 @@
 use Symfony\Component\HttpKernel\EventListener\ProfilerListener;
 use Symfony\Component\HttpKernel\Profiler\FileProfilerStorage;
 use Symfony\Component\HttpKernel\Profiler\Profiler;
+use Symfony\Component\HttpKernel\Profiler\ProfilerStateChecker;
 
 return static function (ContainerConfigurator $container) {
     $container->services()
@@ -56,5 +57,15 @@
         ->set('.virtual_request_stack', VirtualRequestStack::class)
             ->args([service('request_stack')])
             ->public()
+
+        ->set('profiler.state_checker', ProfilerStateChecker::class)
+            ->args([
+                service_locator(['profiler' => service('profiler')->ignoreOnUninitialized()]),
+                param('kernel.runtime_mode.web'),
+            ])
+
+        ->set('profiler.is_disabled_state_checker', 'Closure')
+            ->factory(['Closure', 'fromCallable'])
+            ->args([[service('profiler.state_checker'), 'isProfilerDisabled']])
     ;
 };

Resources/config/validator_debug.php

@@ -20,6 +20,7 @@
             ->decorate('validator', null, 255)
             ->args([
                 service('debug.validator.inner'),
+                service('profiler.is_disabled_state_checker')->nullOnInvalid(),
             ])
             ->tag('kernel.reset', [
                 'method' => 'reset',