Use decoration instead of overriding the service definition [#3476352]

Problem/Motivation

The definitions of the http_middleware.page_cache service is overriden, setting the custom service class AdvancedPageCache.php. This causes this module to not play well with other modules that alter page cache, like page_cache_query_ignore for example.

Steps to reproduce

Proposed resolution

The solution is to use service decoration, this way all the module that wish to alter how page cache works can do so in a layered way.

Remaining tasks

Post a MR.

User interface changes

API changes

Data model changes

Issue fork advanced_page_cache-3476352

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3476352-use-decoration-instead changes, plain diff MR !3
Check out this branch for the first time

Check out existing branch, if you already have it locally