Always instrument ActiveSupport::Cache

The current approach is broken because it uses a thread local value which
means on multi-threaded environments it has to be turned on per thread.
Secondly, ActiveSupport::Notifications does not instrument items when there
are not subscribers so this flag is unnecessary.
We should print a deprecation warning for these methods. Could you add it?