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

Unified Diff: webkit/quota/usage_tracker_unittest.cc

Issue 15950004: [Quota] Wire up limited origin usage retrieving. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix test expectation Created 7 years, 7 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
« webkit/quota/usage_tracker.cc ('K') | « webkit/quota/usage_tracker.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/quota/usage_tracker_unittest.cc
diff --git a/webkit/quota/usage_tracker_unittest.cc b/webkit/quota/usage_tracker_unittest.cc
index 63161262ad8a3776d3b1e1ac95f247e5bcec31aa..412a171ac85faf49f0b8beab60a47226cb4aa10e 100644
--- a/webkit/quota/usage_tracker_unittest.cc
+++ b/webkit/quota/usage_tracker_unittest.cc
@@ -136,6 +136,15 @@ class UsageTrackerTest : public testing::Test {
quota_client_.UpdateUsage(origin, delta);
}
+ void GetGlobalLimitedUsage(int64* limited_usage) {
+ bool done = false;
+ usage_tracker_.GetGlobalLimitedUsage(base::Bind(
+ &DidGetUsage, &done, limited_usage));
+ message_loop_.RunUntilIdle();
+
+ EXPECT_TRUE(done);
+ }
+
void GetGlobalUsage(int64* usage, int64* unlimited_usage) {
bool done = false;
usage_tracker_.GetGlobalUsage(base::Bind(
@@ -261,7 +270,7 @@ TEST_F(UsageTrackerTest, CacheDisabledClientTest) {
GetGlobalUsage(&usage, &unlimited_usage);
GetHostUsage(host, &host_usage);
EXPECT_EQ(400, usage);
- EXPECT_EQ(400, unlimited_usage);
+ EXPECT_EQ(0, unlimited_usage);
EXPECT_EQ(400, host_usage);
SetUsageCacheEnabled(origin, true);
@@ -274,4 +283,42 @@ TEST_F(UsageTrackerTest, CacheDisabledClientTest) {
EXPECT_EQ(500, host_usage);
}
+TEST_F(UsageTrackerTest, LimitedGlobalUsageTest) {
+ const GURL kNormal("http://normal");
+ const GURL kUnlimited("http://unlimited");
+ const GURL kNonCached("http://non_cached");
+ const GURL kNonCachedUnlimited("http://non_cached-unlimited");
+
+ GrantUnlimitedStoragePolicy(kUnlimited);
+ GrantUnlimitedStoragePolicy(kNonCachedUnlimited);
+
+ SetUsageCacheEnabled(kNonCached, false);
+ SetUsageCacheEnabled(kNonCachedUnlimited, false);
+
+ UpdateUsageWithoutNotification(kNormal, 1);
+ UpdateUsageWithoutNotification(kUnlimited, 2);
+ UpdateUsageWithoutNotification(kNonCached, 4);
+ UpdateUsageWithoutNotification(kNonCachedUnlimited, 8);
+
+ int64 limited_usage = 0;
+ int64 total_usage = 0;
+ int64 unlimited_usage = 0;
+
+ GetGlobalLimitedUsage(&limited_usage);
+ GetGlobalUsage(&total_usage, &unlimited_usage);
+ EXPECT_EQ(1 + 4, limited_usage);
+ EXPECT_EQ(1 + 2 + 4 + 8, total_usage);
+ EXPECT_EQ(2 + 8, unlimited_usage);
+
+ UpdateUsageWithoutNotification(kNonCached, 16 - 2);
+ UpdateUsageWithoutNotification(kNonCachedUnlimited, 32 - 8);
+
+ GetGlobalLimitedUsage(&limited_usage);
+ GetGlobalUsage(&total_usage, &unlimited_usage);
+ EXPECT_EQ(1 + 16, limited_usage);
+ EXPECT_EQ(1 + 16 + 4 + 32, total_usage);
+ EXPECT_EQ(2 + 32, unlimited_usage);
+}
+
+
} // namespace quota
« webkit/quota/usage_tracker.cc ('K') | « webkit/quota/usage_tracker.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698