Do not make 1000s of calls to get _deprecated_service_list parameter on every uncached page request

Problem/Motivation

This is a micro-optimisation but we're making thousands of calls to \Drupal\Component\DependencyInjection\Container::get() when we return an uncached page and on every call we do

    if ($this->hasParameter('_deprecated_service_list')) {
      if ($deprecation = $this->getParameter('_deprecated_service_list')[$id] ?? '') {
        @trigger_error($deprecation, E_USER_DEPRECATED);
      }
    }

which we actually only need to do once when we instantiate the service.

Steps to reproduce

Get any service from the container twice.

Proposed resolution

Move the code to after the early returns. On a post request from the autosave module this reduces the calls to Drupal\Component\Dependency…tion\Container::getParameter() from 2890 to 378. Note this is a micro-optimisation because the calls are very cheap.

Remaining tasks

User interface changes

None

Introduced terminology

N/a

API changes

None

Data model changes

None

Release notes snippet

N/a