Remove ContainerAwareTrait from CacheFactory and ChainedFastBackendFactory

This also fails in ServiceProviderTestServiceProvider:

    // Make sure a cached service can be also called in a service provider.
    // https://www.drupal.org/project/drupal/issues/2363351
    /** @var \Drupal\Core\Extension\ModuleHandlerInterface $module_handler */
    $module_handler = $container->get('module_handler');

The module handler ultimately depends on the cache factory, but because of the way the service locator is injected we end up with this error:

1) Drupal\KernelTests\Core\DrupalKernel\DrupalKernelTest::testCompileDIC
Symfony\Component\DependencyInjection\Exception\RuntimeException: Constructing service "cache_factory" from a parent definition is not supported at build time.

As we need cacheable services accessible at container build time, not sure there is a way around this; we might have to inject the entire container instead.

longwave → closed merge request !6246

ChainedFastBackendFactory follows the same pattern. Have not figured out how to work around the issue in #5, so the simplest solution is just to continue injecting the container into both factory services.

I also don't see a way around injecting the whole container in both services.

Tests are green, code changes make sense and are minimal, don't think this needs a CR?

RTBC.

Yeah no CR is needed, no API is changing here really, we are just copying the method and property directly into the class.

catch → credited taraskorpach → .

Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks! Also can't see any need for a CR here.

Please close MR

longwave → closed merge request !6797

Automatically closed - issue fixed for 2 weeks with no activity.