Index: content/browser/appcache/appcache_quota_client_unittest.cc |
diff --git a/content/browser/appcache/appcache_quota_client_unittest.cc b/content/browser/appcache/appcache_quota_client_unittest.cc |
index 88346eec7e61dbe6143773e9f638dfa089f60aac..a453ee1c7739e763c6f0a87a74b00bd22753d631 100644 |
--- a/content/browser/appcache/appcache_quota_client_unittest.cc |
+++ b/content/browser/appcache/appcache_quota_client_unittest.cc |
@@ -16,8 +16,8 @@ |
namespace content { |
// Declared to shorten the line lengths. |
-static const quota::StorageType kTemp = quota::kStorageTypeTemporary; |
-static const quota::StorageType kPerm = quota::kStorageTypePersistent; |
+static const storage::StorageType kTemp = storage::kStorageTypeTemporary; |
+static const storage::StorageType kPerm = storage::kStorageTypePersistent; |
// Base class for our test fixtures. |
class AppCacheQuotaClientTest : public testing::Test { |
@@ -31,85 +31,76 @@ class AppCacheQuotaClientTest : public testing::Test { |
kOriginB("http://host:8000"), |
kOriginOther("http://other"), |
usage_(0), |
- delete_status_(quota::kQuotaStatusUnknown), |
+ delete_status_(storage::kQuotaStatusUnknown), |
num_get_origin_usage_completions_(0), |
num_get_origins_completions_(0), |
num_delete_origins_completions_(0), |
- weak_factory_(this) { |
- } |
+ weak_factory_(this) {} |
- int64 GetOriginUsage( |
- quota::QuotaClient* client, |
- const GURL& origin, |
- quota::StorageType type) { |
+ int64 GetOriginUsage(storage::QuotaClient* client, |
+ const GURL& origin, |
+ storage::StorageType type) { |
usage_ = -1; |
AsyncGetOriginUsage(client, origin, type); |
base::RunLoop().RunUntilIdle(); |
return usage_; |
} |
- const std::set<GURL>& GetOriginsForType( |
- quota::QuotaClient* client, |
- quota::StorageType type) { |
+ const std::set<GURL>& GetOriginsForType(storage::QuotaClient* client, |
+ storage::StorageType type) { |
origins_.clear(); |
AsyncGetOriginsForType(client, type); |
base::RunLoop().RunUntilIdle(); |
return origins_; |
} |
- const std::set<GURL>& GetOriginsForHost( |
- quota::QuotaClient* client, |
- quota::StorageType type, |
- const std::string& host) { |
+ const std::set<GURL>& GetOriginsForHost(storage::QuotaClient* client, |
+ storage::StorageType type, |
+ const std::string& host) { |
origins_.clear(); |
AsyncGetOriginsForHost(client, type, host); |
base::RunLoop().RunUntilIdle(); |
return origins_; |
} |
- quota::QuotaStatusCode DeleteOriginData( |
- quota::QuotaClient* client, |
- quota::StorageType type, |
- const GURL& origin) { |
- delete_status_ = quota::kQuotaStatusUnknown; |
+ storage::QuotaStatusCode DeleteOriginData(storage::QuotaClient* client, |
+ storage::StorageType type, |
+ const GURL& origin) { |
+ delete_status_ = storage::kQuotaStatusUnknown; |
AsyncDeleteOriginData(client, type, origin); |
base::RunLoop().RunUntilIdle(); |
return delete_status_; |
} |
- void AsyncGetOriginUsage( |
- quota::QuotaClient* client, |
- const GURL& origin, |
- quota::StorageType type) { |
+ void AsyncGetOriginUsage(storage::QuotaClient* client, |
+ const GURL& origin, |
+ storage::StorageType type) { |
client->GetOriginUsage( |
origin, type, |
base::Bind(&AppCacheQuotaClientTest::OnGetOriginUsageComplete, |
weak_factory_.GetWeakPtr())); |
} |
- void AsyncGetOriginsForType( |
- quota::QuotaClient* client, |
- quota::StorageType type) { |
+ void AsyncGetOriginsForType(storage::QuotaClient* client, |
+ storage::StorageType type) { |
client->GetOriginsForType( |
type, |
base::Bind(&AppCacheQuotaClientTest::OnGetOriginsComplete, |
weak_factory_.GetWeakPtr())); |
} |
- void AsyncGetOriginsForHost( |
- quota::QuotaClient* client, |
- quota::StorageType type, |
- const std::string& host) { |
+ void AsyncGetOriginsForHost(storage::QuotaClient* client, |
+ storage::StorageType type, |
+ const std::string& host) { |
client->GetOriginsForHost( |
type, host, |
base::Bind(&AppCacheQuotaClientTest::OnGetOriginsComplete, |
weak_factory_.GetWeakPtr())); |
} |
- void AsyncDeleteOriginData( |
- quota::QuotaClient* client, |
- quota::StorageType type, |
- const GURL& origin) { |
+ void AsyncDeleteOriginData(storage::QuotaClient* client, |
+ storage::StorageType type, |
+ const GURL& origin) { |
client->DeleteOriginData( |
origin, type, |
base::Bind(&AppCacheQuotaClientTest::OnDeleteOriginDataComplete, |
@@ -147,7 +138,7 @@ class AppCacheQuotaClientTest : public testing::Test { |
origins_ = origins; |
} |
- void OnDeleteOriginDataComplete(quota::QuotaStatusCode status) { |
+ void OnDeleteOriginDataComplete(storage::QuotaStatusCode status) { |
++num_delete_origins_completions_; |
delete_status_ = status; |
} |
@@ -155,7 +146,7 @@ class AppCacheQuotaClientTest : public testing::Test { |
base::MessageLoop message_loop_; |
int64 usage_; |
std::set<GURL> origins_; |
- quota::QuotaStatusCode delete_status_; |
+ storage::QuotaStatusCode delete_status_; |
int num_get_origin_usage_completions_; |
int num_get_origins_completions_; |
int num_delete_origins_completions_; |
@@ -181,8 +172,8 @@ TEST_F(AppCacheQuotaClientTest, EmptyService) { |
EXPECT_TRUE(GetOriginsForType(client, kPerm).empty()); |
EXPECT_TRUE(GetOriginsForHost(client, kTemp, kOriginA.host()).empty()); |
EXPECT_TRUE(GetOriginsForHost(client, kPerm, kOriginA.host()).empty()); |
- EXPECT_EQ(quota::kQuotaStatusOk, DeleteOriginData(client, kTemp, kOriginA)); |
- EXPECT_EQ(quota::kQuotaStatusOk, DeleteOriginData(client, kPerm, kOriginA)); |
+ EXPECT_EQ(storage::kQuotaStatusOk, DeleteOriginData(client, kTemp, kOriginA)); |
+ EXPECT_EQ(storage::kQuotaStatusOk, DeleteOriginData(client, kPerm, kOriginA)); |
Call_NotifyAppCacheDestroyed(client); |
Call_OnQuotaManagerDestroyed(client); |
@@ -199,9 +190,9 @@ TEST_F(AppCacheQuotaClientTest, NoService) { |
EXPECT_TRUE(GetOriginsForType(client, kPerm).empty()); |
EXPECT_TRUE(GetOriginsForHost(client, kTemp, kOriginA.host()).empty()); |
EXPECT_TRUE(GetOriginsForHost(client, kPerm, kOriginA.host()).empty()); |
- EXPECT_EQ(quota::kQuotaErrorAbort, |
+ EXPECT_EQ(storage::kQuotaErrorAbort, |
DeleteOriginData(client, kTemp, kOriginA)); |
- EXPECT_EQ(quota::kQuotaErrorAbort, |
+ EXPECT_EQ(storage::kQuotaErrorAbort, |
DeleteOriginData(client, kPerm, kOriginA)); |
Call_OnQuotaManagerDestroyed(client); |
@@ -276,17 +267,15 @@ TEST_F(AppCacheQuotaClientTest, DeleteOriginData) { |
// Perm deletions are short circuited in the Client and |
// should not reach the AppCacheServiceImpl. |
- EXPECT_EQ(quota::kQuotaStatusOk, |
- DeleteOriginData(client, kPerm, kOriginA)); |
+ EXPECT_EQ(storage::kQuotaStatusOk, DeleteOriginData(client, kPerm, kOriginA)); |
EXPECT_EQ(0, mock_service_.delete_called_count()); |
- EXPECT_EQ(quota::kQuotaStatusOk, |
- DeleteOriginData(client, kTemp, kOriginA)); |
+ EXPECT_EQ(storage::kQuotaStatusOk, DeleteOriginData(client, kTemp, kOriginA)); |
EXPECT_EQ(1, mock_service_.delete_called_count()); |
mock_service_.set_mock_delete_appcaches_for_origin_result( |
net::ERR_ABORTED); |
- EXPECT_EQ(quota::kQuotaErrorAbort, |
+ EXPECT_EQ(storage::kQuotaErrorAbort, |
DeleteOriginData(client, kTemp, kOriginA)); |
EXPECT_EQ(2, mock_service_.delete_called_count()); |
@@ -368,7 +357,7 @@ TEST_F(AppCacheQuotaClientTest, DestroyServiceWithPending) { |
EXPECT_EQ(3, num_delete_origins_completions_); |
EXPECT_EQ(0, usage_); |
EXPECT_TRUE(origins_.empty()); |
- EXPECT_EQ(quota::kQuotaErrorAbort, delete_status_); |
+ EXPECT_EQ(storage::kQuotaErrorAbort, delete_status_); |
Call_OnQuotaManagerDestroyed(client); |
} |
@@ -421,13 +410,13 @@ TEST_F(AppCacheQuotaClientTest, DestroyWithDeleteInProgress) { |
// Should have been aborted. |
EXPECT_EQ(1, num_delete_origins_completions_); |
- EXPECT_EQ(quota::kQuotaErrorAbort, delete_status_); |
+ EXPECT_EQ(storage::kQuotaErrorAbort, delete_status_); |
// A real completion callback from the service should |
// be dropped if it comes in after NotifyAppCacheDestroyed. |
base::RunLoop().RunUntilIdle(); |
EXPECT_EQ(1, num_delete_origins_completions_); |
- EXPECT_EQ(quota::kQuotaErrorAbort, delete_status_); |
+ EXPECT_EQ(storage::kQuotaErrorAbort, delete_status_); |
Call_OnQuotaManagerDestroyed(client); |
} |