Convert both BookNavigationCacheContext and MenuActiveTrailsCacheContext to use lazy services

Postponing on the possible adoption of native Symfony lazy services in 📌 Deprecate Drupal ProxyBuilder in favor of Symfony lazy services Closed: won't fix .

📌 Deprecate Drupal ProxyBuilder in favor of Symfony lazy services Closed: won't fix is trickier than expected, let's not postpone better on perfect here.

Pipeline is green and CR has the main points.

Almost marked because the syntax in the generated classes seemed off but checking others guess that's the case of how they get generated. And gitlabCI didn't complain.

If this gets in should the scope of 📌 Deprecate Drupal ProxyBuilder in favor of Symfony lazy services Closed: won't fix be expanded?

Committed/pushed to 11.x and cherry-picked to 10.2.x, thanks!

Spokje → closed merge request !5172

Published CR.

This change appears to be causing the issue discussed in 🐛 MenuActiveTrail ($menu_link_manager) must be of type Menu\MenuLinkManagerInterface, DependencyInjection\Container given Active after upgrading to 10.2.0-beta1.

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