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

Unified Diff: ios/chrome/browser/browsing_data/cache_counter.cc

Issue 2623033003: Always show counters in the material design Clear Browsing Data dialog. (Closed)
Patch Set: iOS fix #2 Created 3 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: ios/chrome/browser/browsing_data/cache_counter.cc
diff --git a/ios/chrome/browser/browsing_data/cache_counter.cc b/ios/chrome/browser/browsing_data/cache_counter.cc
index ada6b75652ba5cf7592a3c2a0e40662b938d3e17..c81b4d9343071a769f8646da30467cf693a02896 100644
--- a/ios/chrome/browser/browsing_data/cache_counter.cc
+++ b/ios/chrome/browser/browsing_data/cache_counter.cc
@@ -110,7 +110,7 @@ class IOThreadCacheCounter {
} // namespace
CacheCounter::CacheCounter(web::BrowserState* browser_state)
- : pending_(false), browser_state_(browser_state), weak_ptr_factory_(this) {}
+ : browser_state_(browser_state), weak_ptr_factory_(this) {}
CacheCounter::~CacheCounter() {}
@@ -124,8 +124,6 @@ void CacheCounter::Count() {
// period setting and always request counting for the unbounded time interval.
// It is up to the UI to interpret the results for finite time intervals as
// upper estimates.
- pending_ = true;
-
// IOThreadCacheCounter deletes itself when done.
(new IOThreadCacheCounter(browser_state_->GetRequestContext(),
base::Bind(&CacheCounter::OnCacheSizeCalculated,
@@ -134,8 +132,6 @@ void CacheCounter::Count() {
}
void CacheCounter::OnCacheSizeCalculated(int result_bytes) {
- pending_ = false;
-
// A value less than 0 means a net error code.
if (result_bytes < 0)
return;
« no previous file with comments | « ios/chrome/browser/browsing_data/cache_counter.h ('k') | ios/chrome/browser/browsing_data/cache_counter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698