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

Unified Diff: webkit/browser/quota/mock_quota_manager.cc

Issue 136573007: Quota: Factor out (Mock)QuotaManagerProxy into its own file for readability (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review fix 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
« no previous file with comments | « webkit/browser/quota/mock_quota_manager.h ('k') | webkit/browser/quota/mock_quota_manager_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/browser/quota/mock_quota_manager.cc
diff --git a/webkit/browser/quota/mock_quota_manager.cc b/webkit/browser/quota/mock_quota_manager.cc
index 69e3b0edb95e0bc670fe271ef0a6d123c9def0ca..8f7798ff59124334364516cbdc041ce87daeb735 100644
--- a/webkit/browser/quota/mock_quota_manager.cc
+++ b/webkit/browser/quota/mock_quota_manager.cc
@@ -4,10 +4,6 @@
#include "webkit/browser/quota/mock_quota_manager.h"
-#include <set>
-#include <string>
-#include <vector>
-
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
#include "base/message_loop/message_loop.h"
@@ -32,8 +28,6 @@ MockQuotaManager::OriginInfo::~OriginInfo() {}
MockQuotaManager::StorageInfo::StorageInfo() : usage(0), quota(kint64max) {}
MockQuotaManager::StorageInfo::~StorageInfo() {}
-// MockQuotaManager ----------------------------------------------------------
-
MockQuotaManager::MockQuotaManager(
bool is_incognito,
const base::FilePath& profile_path,
@@ -148,52 +142,4 @@ void MockQuotaManager::DidDeleteOriginData(
callback.Run(status);
}
-// MockQuotaManagerProxy -----------------------------------------------------
-
-MockQuotaManagerProxy::MockQuotaManagerProxy(
- MockQuotaManager* quota_manager,
- base::SingleThreadTaskRunner* task_runner)
- : QuotaManagerProxy(quota_manager, task_runner),
- storage_accessed_count_(0),
- storage_modified_count_(0),
- last_notified_type_(kStorageTypeUnknown),
- last_notified_delta_(0),
- registered_client_(NULL) {}
-
-void MockQuotaManagerProxy::RegisterClient(QuotaClient* client) {
- DCHECK(!registered_client_);
- registered_client_ = client;
-}
-
-void MockQuotaManagerProxy::SimulateQuotaManagerDestroyed() {
- if (registered_client_) {
- // We cannot call this in the destructor as the client (indirectly)
- // holds a refptr of the proxy.
- registered_client_->OnQuotaManagerDestroyed();
- registered_client_ = NULL;
- }
-}
-
-void MockQuotaManagerProxy::NotifyStorageAccessed(
- QuotaClient::ID client_id, const GURL& origin, StorageType type) {
- ++storage_accessed_count_;
- last_notified_origin_ = origin;
- last_notified_type_ = type;
-}
-
-void MockQuotaManagerProxy::NotifyStorageModified(
- QuotaClient::ID client_id, const GURL& origin,
- StorageType type, int64 delta) {
- ++storage_modified_count_;
- last_notified_origin_ = origin;
- last_notified_type_ = type;
- last_notified_delta_ = delta;
- if (mock_manager())
- mock_manager()->UpdateUsage(origin, type, delta);
-}
-
-MockQuotaManagerProxy::~MockQuotaManagerProxy() {
- DCHECK(!registered_client_);
-}
-
} // namespace quota
« no previous file with comments | « webkit/browser/quota/mock_quota_manager.h ('k') | webkit/browser/quota/mock_quota_manager_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698