Merge branch '3.3' into 3.4

* 3.3:
  [DI] use assertStringEqualsFile when possible
  [VarDumper] Adapt to php 7.2 changes
  [DI] Fix using private services in expressions
  [Form][TwigBridge] Don't render _method in form_rest() for a child form
  [Form] Static call TimezoneType::getTimezones
  Removed references for non existent validator constraints
  Suggest using quotes instead of Yaml::PARSE_KEYS_AS_STRINGS
  [DI] Fix test
  [Cache] Handle unserialization failures for Memcached
  Remove unused prop + added @deprecated
  Remove unused mocks/vars
  [DoctrineBridge][PropertyInfo] Added support for Doctrine Embeddables
  [Validator] Fix IbanValidator for ukrainian IBANs
  Router: allow HEAD method to be defined first
  [WebProfilerBundle] Display trace and context in the logger profiler
  Fixing a bug where if a core class was autowired, autowiring tried to autowire optional args as if they were required

Author: nicolas-grekas

Tests/Command/RouterDebugCommandTest.php

@@ -71,10 +71,6 @@ private function getKernel()
             ->will($this->returnValue($routeCollection))
         ;
 
-        $loader = $this->getMockBuilder('Symfony\Bundle\FrameworkBundle\Routing\DelegatingLoader')
-            ->disableOriginalConstructor()
-            ->getMock();
-
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerInterface')->getMock();
         $container
             ->expects($this->atLeastOnce())

Tests/Command/RouterMatchCommandTest.php

@@ -70,10 +70,6 @@ private function getKernel()
             ->will($this->returnValue($requestContext))
         ;
 
-        $loader = $this->getMockBuilder('Symfony\Bundle\FrameworkBundle\Routing\DelegatingLoader')
-            ->disableOriginalConstructor()
-            ->getMock();
-
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerInterface')->getMock();
         $container
             ->expects($this->atLeastOnce())

Tests/Functional/CachePoolClearCommandTest.php

@@ -19,8 +19,6 @@
  */
 class CachePoolClearCommandTest extends WebTestCase
 {
-    private $application;
-
     protected function setUp()
     {
         static::bootKernel(array('test_case' => 'CachePoolClear', 'root_config' => 'config.yml'));

Tests/Functional/ConfigDebugCommandTest.php

@@ -61,7 +61,7 @@ public function testParametersValuesAreResolved()
     public function testDumpUndefinedBundleOption()
     {
         $tester = $this->createCommandTester();
-        $ret = $tester->execute(array('name' => 'TestBundle', 'path' => 'foo'));
+        $tester->execute(array('name' => 'TestBundle', 'path' => 'foo'));
 
         $this->assertContains('Unable to find configuration for "test.foo"', $tester->getDisplay());
     }