Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(139)

Unified Diff: trunk/src/chrome/browser/extensions/updater/local_extension_cache.cc

Issue 139173009: Revert 247760 "Add UMA stats for ExtensionCache" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/chrome/browser/extensions/updater/local_extension_cache.cc
===================================================================
--- trunk/src/chrome/browser/extensions/updater/local_extension_cache.cc (revision 247820)
+++ trunk/src/chrome/browser/extensions/updater/local_extension_cache.cc (working copy)
@@ -29,7 +29,7 @@
LocalExtensionCache::LocalExtensionCache(
const base::FilePath& cache_dir,
- uint64 max_cache_size,
+ size_t max_cache_size,
const base::TimeDelta& max_cache_age,
const scoped_refptr<base::SequencedTaskRunner>& backend_task_runner)
: cache_dir_(cache_dir),
@@ -151,21 +151,6 @@
return true;
}
-bool LocalExtensionCache::GetStatistics(uint64* cache_size,
- size_t* extensions_count) {
- if (state_ != kReady)
- return false;
-
- *cache_size = 0;
- for (CacheMap::iterator it = cached_extensions_.begin();
- it != cached_extensions_.end(); ++it) {
- *cache_size += it->second.size;
- }
- *extensions_count = cached_extensions_.size();
-
- return true;
-}
-
void LocalExtensionCache::SetCacheStatusPollingDelayForTests(
const base::TimeDelta& delay) {
cache_status_polling_delay_ = delay;
@@ -440,7 +425,7 @@
std::vector<CacheMap::iterator> items;
items.reserve(cached_extensions_.size());
- uint64_t total_size = 0;
+ size_t total_size = 0;
for (CacheMap::iterator it = cached_extensions_.begin();
it != cached_extensions_.end(); ++it) {
items.push_back(it);
« no previous file with comments | « trunk/src/chrome/browser/extensions/updater/local_extension_cache.h ('k') | trunk/src/tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698