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

Unified Diff: webkit/quota/mock_quota_manager.cc

Issue 8070001: Use base::Callback in Quota related code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 9 years, 2 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
« no previous file with comments | « webkit/quota/mock_quota_manager.h ('k') | webkit/quota/mock_quota_manager_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/quota/mock_quota_manager.cc
diff --git a/webkit/quota/mock_quota_manager.cc b/webkit/quota/mock_quota_manager.cc
index 67f1ee3ebf9d4ad91de3aaf4c5448aada155a41e..52d27351e60015b0483889370e9e2827c657a1c3 100644
--- a/webkit/quota/mock_quota_manager.cc
+++ b/webkit/quota/mock_quota_manager.cc
@@ -23,7 +23,7 @@ class MockQuotaManager::GetModifiedSinceTask : public QuotaThreadTask {
GetModifiedSinceTask(MockQuotaManager* manager,
const std::set<GURL>& origins,
StorageType type,
- GetOriginsCallback* callback)
+ const GetOriginsCallback& callback)
: QuotaThreadTask(manager, manager->io_thread_),
origins_(origins),
type_(type),
@@ -33,17 +33,17 @@ class MockQuotaManager::GetModifiedSinceTask : public QuotaThreadTask {
virtual void RunOnTargetThread() OVERRIDE {}
virtual void Completed() OVERRIDE {
- callback_->Run(origins_, type_);
+ callback_.Run(origins_, type_);
}
virtual void Aborted() OVERRIDE {
- callback_->Run(std::set<GURL>(), type_);
+ callback_.Run(std::set<GURL>(), type_);
}
private:
std::set<GURL> origins_;
StorageType type_;
- scoped_ptr<GetOriginsCallback> callback_;
+ GetOriginsCallback callback_;
DISALLOW_COPY_AND_ASSIGN(GetModifiedSinceTask);
};
@@ -51,7 +51,7 @@ class MockQuotaManager::GetModifiedSinceTask : public QuotaThreadTask {
class MockQuotaManager::DeleteOriginDataTask : public QuotaThreadTask {
public:
DeleteOriginDataTask(MockQuotaManager* manager,
- StatusCallback* callback)
+ const StatusCallback& callback)
: QuotaThreadTask(manager, manager->io_thread_),
callback_(callback) {}
@@ -59,15 +59,15 @@ class MockQuotaManager::DeleteOriginDataTask : public QuotaThreadTask {
virtual void RunOnTargetThread() OVERRIDE {}
virtual void Completed() OVERRIDE {
- callback_->Run(quota::kQuotaStatusOk);
+ callback_.Run(quota::kQuotaStatusOk);
}
virtual void Aborted() OVERRIDE {
- callback_->Run(quota::kQuotaErrorAbort);
+ callback_.Run(quota::kQuotaErrorAbort);
}
private:
- scoped_ptr<StatusCallback> callback_;
+ StatusCallback callback_;
DISALLOW_COPY_AND_ASSIGN(DeleteOriginDataTask);
};
@@ -110,8 +110,10 @@ bool MockQuotaManager::OriginHasData(const GURL& origin,
return false;
}
-void MockQuotaManager::GetOriginsModifiedSince(StorageType type,
- base::Time modified_since, GetOriginsCallback* callback) {
+void MockQuotaManager::GetOriginsModifiedSince(
+ StorageType type,
+ base::Time modified_since,
+ const GetOriginsCallback& callback) {
std::set<GURL> origins_to_return;
for (std::vector<OriginInfo>::const_iterator current = origins_.begin();
current != origins_.end();
@@ -124,7 +126,7 @@ void MockQuotaManager::GetOriginsModifiedSince(StorageType type,
}
void MockQuotaManager::DeleteOriginData(const GURL& origin, StorageType type,
- StatusCallback* callback) {
+ const StatusCallback& callback) {
for (std::vector<OriginInfo>::iterator current = origins_.begin();
current != origins_.end();
++current) {
« no previous file with comments | « webkit/quota/mock_quota_manager.h ('k') | webkit/quota/mock_quota_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698