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

Unified Diff: webkit/quota/quota_manager.cc

Issue 14237003: Rename SpecialStoragePolicy::IsInstalledApp() to CanQueryDiskSize() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 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/quota_manager.h ('k') | webkit/quota/quota_manager_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/quota/quota_manager.cc
diff --git a/webkit/quota/quota_manager.cc b/webkit/quota/quota_manager.cc
index 2c810a727bbfdfb2a44b7d0ec0bff20a3e9b5161..93462ce2976ff0804c0b1d2e1a7dae2365547b23 100644
--- a/webkit/quota/quota_manager.cc
+++ b/webkit/quota/quota_manager.cc
@@ -185,7 +185,7 @@ const int QuotaManager::kEvictionIntervalInMilliSeconds =
// and by multiple apps.
int64 QuotaManager::kSyncableStorageDefaultHostQuota = 500 * kMBytes;
-int64 CalculateQuotaForInstalledApp(
+int64 CalculateQuotaWithDiskSpace(
int64 available_disk_space, int64 usage, int64 quota) {
if (available_disk_space < QuotaManager::kMinimumPreserveForSystem ||
quota < usage) {
@@ -204,15 +204,15 @@ int64 CalculateQuotaForInstalledApp(
void CallGetUsageAndQuotaCallback(
const QuotaManager::GetUsageAndQuotaCallback& callback,
bool unlimited,
- bool is_installed_app,
+ bool can_query_disk_size,
QuotaStatusCode status,
const QuotaAndUsage& quota_and_usage) {
// Regular limited case.
if (!unlimited) {
- if (is_installed_app) {
+ if (can_query_disk_size) {
// Cap the quota by the available disk space.
callback.Run(status, quota_and_usage.usage,
- CalculateQuotaForInstalledApp(
+ CalculateQuotaWithDiskSpace(
quota_and_usage.available_disk_space,
quota_and_usage.usage,
quota_and_usage.quota));
@@ -224,11 +224,12 @@ void CallGetUsageAndQuotaCallback(
int64 usage = quota_and_usage.unlimited_usage;
- // Unlimited case: this must be only for installed-app and extensions,
+ // Unlimited case: this must be only for apps with unlimitedStorage permission
// or only when --unlimited-storage flag is given.
- // We return the available disk space (minus kMinimumPreserveForSystem).
+ // We assume we can expose the disk size for them and return the available
+ // disk space (minus kMinimumPreserveForSystem).
callback.Run(status, usage,
- CalculateQuotaForInstalledApp(
+ CalculateQuotaWithDiskSpace(
quota_and_usage.available_disk_space,
usage, QuotaManager::kNoLimit));
}
@@ -967,7 +968,7 @@ void QuotaManager::GetUsageAndQuotaForWebApps(
GetUsageAndQuotaInternal(
origin, type, false /* global */,
base::Bind(&CallGetUsageAndQuotaCallback, callback,
- IsStorageUnlimited(origin, type), IsInstalledApp(origin)));
+ IsStorageUnlimited(origin, type), CanQueryDiskSize(origin)));
}
void QuotaManager::GetUsageAndQuota(
@@ -977,7 +978,7 @@ void QuotaManager::GetUsageAndQuota(
if (IsStorageUnlimited(origin, type)) {
CallGetUsageAndQuotaCallback(
- callback, false, IsInstalledApp(origin),
+ callback, false, CanQueryDiskSize(origin),
kQuotaStatusOk, QuotaAndUsage::CreateForUnlimitedStorage());
return;
}
« no previous file with comments | « webkit/quota/quota_manager.h ('k') | webkit/quota/quota_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698