Merge branch '4.4' into 5.4

* 4.4:
  [HttpKernel] Fix message for unresovable arguments of invokable controllers
  [DependencyInjection] Process bindings in ServiceLocatorTagPass

Author: fabpot

Compiler/ServiceLocatorTagPass.php

@@ -40,6 +40,10 @@ protected function processValue($value, bool $isRoot = false)
             return self::register($this->container, $value->getValues());
         }
 
+        if ($value instanceof Definition) {
+            $value->setBindings(parent::processValue($value->getBindings()));
+        }
+
         if (!$value instanceof Definition || !$value->hasTag('container.service_locator')) {
             return parent::processValue($value, $isRoot);
         }

Tests/Compiler/ServiceLocatorTagPassTest.php

@@ -214,6 +214,18 @@ public function testDefinitionOrderIsTheSame()
 
         static::assertSame(['service-2', 'service-1'], array_keys($factories));
     }
+
+    public function testBindingsAreProcessed()
+    {
+        $container = new ContainerBuilder();
+
+        $definition = $container->register('foo')
+            ->setBindings(['foo' => new ServiceLocatorArgument()]);
+
+        (new ServiceLocatorTagPass())->process($container);
+
+        $this->assertInstanceOf(Reference::class, $definition->getBindings()['foo']->getValues()[0]);
+    }
 }
 
 class Locator