Correct visibility of getPluginDefinition function in ContextAwarePluginTrait

Problem/Motivation

The ContextAwarePluginTrait defines the following (source):

  /**
   * {@inheritdoc}
   */
  abstract protected function getPluginDefinition();

But the PluginInspectionInterface defines this as a public function (source):

  /**
   * Gets the definition of the plugin implementation.
   *
   * @return array
   *   The plugin definition, as returned by the discovery object used by the
   *   plugin manager.
   */
  public function getPluginDefinition();

My IDE highlighted this error when creating a plugin that extends the ConditionPluginBase class (which uses the ContextAwarePluginTrait). Changing the function visibility to be public clears the error, but is there a special reason this function was private? Maybe because it was abstract inside the trait?

I tried to search for an existing issue (with this search) but didn't find anything; it seems like this trait was only added recently via: #2273381: Convert ContextAwarePluginBase to traits

In that issue I didn't see any discussion around the visibility of the getPluginDefinition method - it seems like it was recommended as a abstract protected getPluginDefinition in this comment, and that was it: https://www.drupal.org/project/drupal/issues/2273381#comment-13712658

Steps to reproduce

N/A

Proposed resolution

Change the abstract function visibility from protected to public.

Remaining tasks

N/A - I don't think this would be breaking?

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A