Deprecate NodeStorageInterface::revisionIds in favor of entity query

Problem/Motivation

In #2957425: Allow the inline creation of non-reusable Custom Blocks in the layout builder → we are having to get all the revision IDs for entity that we just know is revisionable.

We have generic code

   $entity_type = $this->entityTypeManager->getDefinition($entity->getEntityTypeId());
    if ($revision_table = $entity_type->getRevisionTable()) {
      $query = $this->database->select($revision_table);
      $query->condition($entity_type->getKey('id'), $entity->id());
      $query->fields($revision_table, [$entity_type->getKey('revision')]);
      $query->orderBy($entity_type->getKey('revision'), 'DESC');
      return $query->execute()->fetchCol();
    }
    return [];

Proposed resolution

That should work for any revisionable entity. Can we move NodeStorageInterface::revisionIds on up to RevisionableStorageInterface and then implement this generic logic in \Drupal\Core\Entity\Sql\SqlContentEntityStorage?

Remaining tasks

Is this ok with BC policy.
Write patch
Tests

User interface changes

None

API changes

RevisionableStorageInterface would have ::revisionIds()

Data model changes

NOne