Remove ResourceConfig::getGlobalConfig(), and stop injecting the config factory service in ResourceConfig value objects

.

1. +++ b/src/Configuration/ResourceConfig.php
   @@ -17,13 +17,6 @@ use Drupal\Core\Entity\EntityTypeManagerInterface;
   -  protected $globalConfig;
   
   @@ -68,13 +61,6 @@ class ResourceConfig implements ResourceConfigInterface {
   -  public function getGlobalConfig() {
   -    return $this->globalConfig;
   -  }
   

   This is what's key.

2. +++ b/src/Configuration/ResourceConfig.php
   @@ -148,13 +134,10 @@ class ResourceConfig implements ResourceConfigInterface {
   -  public function __construct(ConfigFactoryInterface $config_factory, EntityTypeManagerInterface $entity_type_manager) {
   -    $this->globalConfig = $config_factory->get('jsonapi.resource_info');
   +  public function __construct(EntityTypeManagerInterface $entity_type_manager) {
   

   This is what follows: ResourceConfig objects no longer need the config factory to be injected.

3. +++ b/src/Configuration/ResourceManager.php
   @@ -23,13 +23,6 @@ class ResourceManager implements ResourceManagerInterface {
   -  protected $configFactory;
   
   @@ -37,13 +30,6 @@ class ResourceManager implements ResourceManagerInterface {
   -  protected $moduleHandler;
   
   @@ -57,16 +43,10 @@ class ResourceManager implements ResourceManagerInterface {
   -    $this->configFactory = $config_factory;
   -    $this->moduleHandler = $module_handler;
      }
   

   Turns out we also no longer need to inject the module handler service.

To be more specific, this is blocking #2841050: Make ResourceConfig an actual value object: immutable, with no services injected.

Also, this is a net reduction: 8 files changed, 6 insertions, 58 deletions :)

Yay! Now on to #2841050: Make ResourceConfig an actual value object: immutable, with no services injected, which is also green :)