Change ViewsData to use the default cache bin instead of discovery

Problem/Motivation

Bug because this is a performance problem.

See #2765271: Rationalize use of the 'discovery' cache bin, since it's stored in the limited size APCu by default for background and data. views_data is in my case 3MB of data, storing that in apcu is not a good idea. We also don't need it on most requests.

Proposed resolution

Two options:

1. Easy: Completely move all entries to the data cache bin. Do so by changing the service definition for the token service and pass in cache.data instead of cache.discovery.
2. A bit more complicated: Add data as a second cache bin, store the full cache in there, the tables still in discovery.

I would suggest we start with 1. On my pretty big site with empty render cache, my frontpage loaded 19 tables from cache. I didn't count but I expect that e.g. render cache did a lot more cache gets there.

Remaining tasks

User interface changes

API changes

Data model changes