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.
I think it would be good to return `true` here.