Sounds good. 

Wondering if I should return a defined cache object type, rather than object? It will have have string, date, timespan and object. I think this would make testing easier.

Christophe Gijbels <notifications@github.com> wrote:
>> @@ -31,13 +31,22 @@ public override object GetData(ITabContext
>context)
>>                  // if more properties want to be displayed in the
>future, here's a list of properties available
>>                  //     Value - UtcCreated - State - UtcExpires -
>SlidingExpiration - ExpiresBucket - ExpiresEntryRef 
>>                  //     UsageBucket - UsageEntryRef -
>UtcLastUsageUpdate - Dependency - Key - IsOutputCache - IsPublic
>> -                items.Add(new { 
>> -                    Key = key,
>> -                    Value =
>Serialization.GetValueSafe(cacheEnumerator.Value),
>> -                    Created = GetCacheItemProperty(key,
>"UtcCreated"),
>> -                    Expiry = GetCacheItemProperty(key,
>"UtcExpires"),
>> -                    SlidingExpiration = GetCacheItemProperty(key,
>"SlidingExpiration")
>> -                });
>> +                try
>
>I see what you mean, and we might indeed give the user that expression
>in case he/she would ever see that evicted notice
>
>So maybe it is indeed better to leave the whole item out of the
>results.
>
>Besides that I do wonder if we should catch a `NullReferenceException`
>and _swallow_ it, since the `NullReferenceException` may also occur for
>another reason than the cache item being evicted from the Cache.
>
>So I was thinking, if we first create a method `TryGetCacheEntry(key,
>out object cacheEntry)` that returns, if possible, the corresponding
>CacheEntry, then we would prevent an exception from being thrown and
>have a performance improvement since the cache entry is only requested
>once instead of 3 times now, once for each property (and we handle the
>case where the item gets evicted between calls to the
>`GetCacheItemProperty` for the same key.)
>
>what do you think?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/Glimpse/Glimpse/pull/652/files#r7924067