| Index: chrome/browser/extensions/extension_special_storage_policy.cc
|
| diff --git a/chrome/browser/extensions/extension_special_storage_policy.cc b/chrome/browser/extensions/extension_special_storage_policy.cc
|
| index 95e0719eeaae0de24292da927d0474132e6cce82..7f0ba19676b4d83ffbefb89ccf74d0939c2ecb91 100644
|
| --- a/chrome/browser/extensions/extension_special_storage_policy.cc
|
| +++ b/chrome/browser/extensions/extension_special_storage_policy.cc
|
| @@ -15,6 +15,7 @@
|
| #include "base/stl_util.h"
|
| #include "base/strings/utf_string_conversions.h"
|
| #include "chrome/browser/content_settings/cookie_settings_factory.h"
|
| +#include "chrome/browser/permissions/permission_manager.h"
|
| #include "chrome/common/chrome_switches.h"
|
| #include "chrome/common/extensions/manifest_handlers/app_launch_info.h"
|
| #include "chrome/common/url_constants.h"
|
| @@ -23,6 +24,7 @@
|
| #include "components/content_settings/core/common/content_settings_types.h"
|
| #include "content/public/browser/browser_context.h"
|
| #include "content/public/browser/browser_thread.h"
|
| +#include "content/public/browser/permission_type.h"
|
| #include "content/public/browser/storage_partition.h"
|
| #include "content/public/common/url_constants.h"
|
| #include "extensions/common/constants.h"
|
| @@ -60,10 +62,10 @@ void LogHostedAppUnlimitedStorageUsage(
|
| GURL launch_url =
|
| extensions::AppLaunchInfo::GetLaunchWebURL(extension.get()).GetOrigin();
|
| content::StoragePartition* partition =
|
| - browser_context ? // |browser_context| can be NULL in unittests.
|
| + browser_context ? // |browser_context| can be nullptr in unittests.
|
| content::BrowserContext::GetStoragePartitionForSite(browser_context,
|
| launch_url) :
|
| - NULL;
|
| + nullptr;
|
| if (partition) {
|
| // We only have to query for kStorageTypePersistent data usage, because apps
|
| // cannot ask for any more temporary storage, according to
|
| @@ -79,9 +81,11 @@ void LogHostedAppUnlimitedStorageUsage(
|
|
|
| } // namespace
|
|
|
| -ExtensionSpecialStoragePolicy::ExtensionSpecialStoragePolicy(
|
| - content_settings::CookieSettings* cookie_settings)
|
| - : cookie_settings_(cookie_settings) {
|
| +ExtensionSpecialStoragePolicy::ExtensionSpecialStoragePolicy(Profile* profile)
|
| + : profile_(profile) {
|
| + if (profile_) {
|
| + cookie_settings_ = CookieSettingsFactory::GetForProfile(profile_).get();
|
| + }
|
| }
|
|
|
| ExtensionSpecialStoragePolicy::~ExtensionSpecialStoragePolicy() {}
|
| @@ -109,7 +113,7 @@ bool ExtensionSpecialStoragePolicy::IsStorageUnlimited(const GURL& origin) {
|
| }
|
|
|
| bool ExtensionSpecialStoragePolicy::IsStorageSessionOnly(const GURL& origin) {
|
| - if (cookie_settings_.get() == NULL)
|
| + if (cookie_settings_ == nullptr)
|
| return false;
|
| return cookie_settings_->IsCookieSessionOnly(origin);
|
| }
|
| @@ -120,9 +124,9 @@ bool ExtensionSpecialStoragePolicy::CanQueryDiskSize(const GURL& origin) {
|
| }
|
|
|
| bool ExtensionSpecialStoragePolicy::HasSessionOnlyOrigins() {
|
| - if (cookie_settings_.get() == NULL)
|
| + if (cookie_settings_ == nullptr)
|
| return false;
|
| - if (cookie_settings_->GetDefaultCookieSetting(NULL) ==
|
| + if (cookie_settings_->GetDefaultCookieSetting(nullptr) ==
|
| CONTENT_SETTING_SESSION_ONLY)
|
| return true;
|
| ContentSettingsForOneType entries;
|
| @@ -140,7 +144,10 @@ bool ExtensionSpecialStoragePolicy::HasIsolatedStorage(const GURL& origin) {
|
| }
|
|
|
| bool ExtensionSpecialStoragePolicy::IsStorageDurable(const GURL& origin) {
|
| - return cookie_settings_->IsStorageDurable(origin);
|
| + blink::mojom::PermissionStatus status =
|
| + PermissionManager::Get(profile_)->GetPermissionStatus(
|
| + content::PermissionType::DURABLE_STORAGE, origin, origin);
|
| + return status == blink::mojom::PermissionStatus::GRANTED;
|
| }
|
|
|
| bool ExtensionSpecialStoragePolicy::NeedsProtection(
|
|
|