Add cache tags to Views data and plugin definition cache entries

Why would ever someone invalidate extension => 'views'?

Oh I see

This is in line with what LibraryDiscovery does:

    $this->cache->set('library:info:' . $extension, $information, Cache::PERMANENT, array(
      'extension' => array(TRUE, $extension),
      'library_info' => array(TRUE),
    ));

RTBC +1

+++ b/core/modules/views/lib/Drupal/views/Plugin/ViewsHandlerManager.php
@@ -57,7 +57,7 @@ public function __construct($handler_type, \Traversable $namespaces, ViewsData $
-    $this->setCacheBackend($cache_backend, $language_manager, "views:$handler_type");
+    $this->setCacheBackend($cache_backend, $language_manager, "views:$handler_type", array('extension' => array(TRUE, 'views')));

+++ b/core/modules/views/lib/Drupal/views/Plugin/ViewsPluginManager.php
@@ -44,7 +44,7 @@ public function __construct($type, \Traversable $namespaces, CacheBackendInterfa
-    $this->setCacheBackend($cache_backend, $language_manager, "views:{$type}_plugins");
+    $this->setCacheBackend($cache_backend, $language_manager, "views:{$type}_plugins", array('extension' => array(TRUE, 'views')));

On the longrun we probably want to store all of that information in one cache entry , but this is work for another issue. You need all of them anyway.

And yes, we ideally do want to consolidate plugins into a single cache entry. I have been speaking to msonnabaum about this too.

Afaik this was was worked on in some other issue.

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.php
@@ -165,7 +166,8 @@ public function initDisplay(ViewExecutable $view, array &$display, array &$optio
+          $id = $this->view->storage->id();
+          \Drupal::cache('data')->set($cid, $this->options, Cache::PERMANENT, array('extension' => array(TRUE, 'views'), 'view' => array($id => $id)));

array(TRUE, 'views') is a little bit odd, can you elobarete why this is needed?

Oh I see, thank you!

Committed cce2380 and pushed to 8.x. Thanks!