Move bulk form data definition to their EntityViewsData

I was reviewing this and this is what views module does on behalf of any module that defines an entity and has actions associated with it (in views.views.inc):

    $data[$entity_info->getBaseTable()][$entity_type . '_bulk_form'] = array(
      'title' => t('Bulk update'),
      'help' => t('Allows users to apply an action to one or more items.'),
      'field' => array(
        'id' => 'bulk_form',
      ),
    );

Couldn't we just remove the particular node/user/* integrations then try to override that without adding much, honestly (in NodeViewsData.php):

$data['node']['node_bulk_form'] = array(
      'title' => t('Node operations bulk form'),
      'help' => t('Add a form element that lets you run operations on multiple nodes.'),
      'field' => array(
        'id' => 'node_bulk_form',
      ),
);

Couldn't we just get rid of the particular implementations and leave just the generic view implementation, maybe with one change, adding the entity_type to the id of the field: 'id' => 'bulk_form', would be 'id' => $entity_type . '_bulk_form',

Ok, got it @dawehner, thanks.

Back to the original issue, tracing this comment, if I've understood it correctly:

this is a bit problematic now ... given that views_views_data() might run afterwards

It is actually views_views_data who invokes the *ViewsData.php plugins:

  // Registers views data for the entity itself.
  foreach (\Drupal::entityManager()->getDefinitions() as $entity_type_id => $entity_type) {
    if ($entity_type->hasHandlerClass('views_data')) {
      /** @var \Drupal\views\EntityViewsDataInterface $views_data */
      $views_data = \Drupal::entityManager()->getHandler($entity_type_id, 'views_data');
      $data = NestedArray::mergeDeep($data, $views_data->getViewsData());
    }
  }

So whatever we've got in views_view_data will come before the $views_data->getViewsData() call to the plugin.

If you add a test with something module_set_weight('node', 1000);, the order of execution is respected, views will always run first, I honestly don't see any value on adding this to the patch, setting back to needs review.