Remove hook_node_access() in favor of hook_ENTITY_TYPE_access()

#2095125: Use access constants in every access control context has been committed.

Tagging.

The return value uses NodeInterface, and so does the $node parameter. People won't have to look further, which is a possibility when we document the return value using AccessInterface.

I thought this would be a DX improvement, but if others agree, I don't mind documenting the return value using AccessInterface instead.

2: drupal_2112023_2.patch queued for re-testing.

Re-roll because of #2108829: Make AccessManager stricter with values returned from access checkers.

7: drupal_2112023_7.patch queued for re-testing.

7: drupal_2112023_7.patch queued for re-testing.

Now we have hook_ENTITY_TYPE_access(), hook_node_access() is nothing more than a particular version of that hook. As such, remove any reference to it. Replacement of the NODE_ACCESS_* constants with those of \Drupal\Core\Access\AccessInterface remains part of this issue, as they were only ever used for hook_node_acces() anyway.

Marking as duplicate of #2151219: Entity and field access hooks must use the AccessInterface constants.