$cacheFactory.remove() throws exception when the key does not exists

[mlegenhausen]

Instead it should simply return.

TypeError: Cannot read property 'n' of undefined
    at Object.caches.(anonymous function).remove (http://localhost:8080/js/lib/angular/angular.js:3318:24)

Angular Version 1.0.2

[pkozlowski-opensource]

Reproduce scenario: http://jsfiddle.net/MC3wm/ (error in the console).

It is probably a good idea to ignore removal of non-existing values (that is, don't throw exceptions in this case). Or at least provide a more descriptive error message.

[jtymes]

@pkozlowski-opensource I saw your comment come through my email but it seems to have disappeared. After you mentioned it, I did a few tests with putting in 0 or false in the cache but it seemed to work alright. I can change it over to use isUndefined() instead if you think that would be a safer route.

[pkozlowski-opensource]

@jtymes Ignore my comment, I've misread your code. The fix looks good. Travis-CI is failing atm but I'm not sure if this is linked to your fix.

[mlegenhausen]

Looks good, cause there the only results of var lruEntry = lruHash[key]; can be an object or undefined. Where the first case will be always true and the seconds always false.