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

Unified Diff: chrome/browser/extensions/system_info_provider.h

Issue 12084017: [SystemInfo API] Implement systemInfo.storage.onAvailableCapacityChanged event (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Add a stub file to pass building on Android Created 7 years, 10 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: chrome/browser/extensions/system_info_provider.h
diff --git a/chrome/browser/extensions/system_info_provider.h b/chrome/browser/extensions/system_info_provider.h
index 558905ad9fec8f372f50c745dafe2d39e3a38dca..fd650eb3f433c983cc346ef7c08cb07efe54c5b5 100644
--- a/chrome/browser/extensions/system_info_provider.h
+++ b/chrome/browser/extensions/system_info_provider.h
@@ -76,23 +76,21 @@ class SystemInfoProvider
worker_pool->PostSequencedWorkerTaskWithShutdownBehavior(
worker_pool_token_,
FROM_HERE,
- base::Bind(&SystemInfoProvider<T>::QueryOnWorkerPool,
- base::Unretained(this)),
+ base::Bind(&SystemInfoProvider<T>::QueryOnWorkerPool, this),
base::SequencedWorkerPool::CONTINUE_ON_SHUTDOWN);
}
+ protected:
// Query the system information synchronously and output the result to the
// |info| parameter. The |info| contents MUST be reset firstly in its
// platform specific implementation. Return true if it succeeds, otherwise
// false is returned.
virtual bool QueryInfo(T* info) = 0;
- protected:
virtual void QueryOnWorkerPool() {
bool success = QueryInfo(&info_);
content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
- base::Bind(&SystemInfoProvider<T>::OnQueryCompleted,
- base::Unretained(this), success));
+ base::Bind(&SystemInfoProvider<T>::OnQueryCompleted, this, success));
}
// Called on UI thread. The |success| parameter means whether it succeeds

Powered by Google App Engine
This is Rietveld 408576698