SiteService.getSites() removes fully cached sites.

I was recently looking at a performance problem where it looked like some code was loading all of a user's sites from the DB. The code in question wasn't calling SiteService.getUserSites() but instead was calling SiteService.getSites() and asking for all sites a user has access to. This falls through to the DB which loads all the Site IDs but, then it checks to see if any of these site IDs are cached (lots normally are), but then attempts to load them all anyway. The really bad part of this is that when loading Sites it is just a partial site BaseSite.isFullyLoaded() returns false and these sites are all put into the cache, replacing sites that were already in the site and already fully loaded.