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

Unified Diff: chrome/browser/browsing_data_helper.cc

Issue 10413072: Teaching BrowsingDataRemover how to delete application data. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Feedback. Created 8 years, 6 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
Index: chrome/browser/browsing_data_helper.cc
diff --git a/chrome/browser/browsing_data_helper.cc b/chrome/browser/browsing_data_helper.cc
index 06de66505d76373f9853415865882a84127349d5..8e586d8134ff5f6dee2bc602a31b4d41cd8069b2 100644
--- a/chrome/browser/browsing_data_helper.cc
+++ b/chrome/browser/browsing_data_helper.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/utf_string_conversions.h"
+#include "chrome/browser/extensions/extension_special_storage_policy.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/url_constants.h"
#include "content/public/browser/child_process_security_policy.h"
@@ -55,3 +56,26 @@ bool BrowsingDataHelper::IsExtensionScheme(const WebKit::WebString& scheme) {
bool BrowsingDataHelper::HasExtensionScheme(const GURL& origin) {
return BrowsingDataHelper::IsExtensionScheme(origin.scheme());
}
+
+// Static
+bool BrowsingDataHelper::DoesOriginMatchMask(const GURL& origin,
+ int origin_set_mask, ExtensionSpecialStoragePolicy* policy) {
+ // Packaged apps and extensions match iff EXTENSION.
+ if (BrowsingDataHelper::HasExtensionScheme(origin.GetOrigin()) &&
+ origin_set_mask & EXTENSION)
+ return true;
+
+ // If a websafe origin is unprotected, it matches iff UNPROTECTED_WEB.
+ if (!policy->IsStorageProtected(origin.GetOrigin()) &&
+ BrowsingDataHelper::HasWebScheme(origin.GetOrigin()) &&
+ origin_set_mask & UNPROTECTED_WEB)
+ return true;
+
+ // Hosted applications (protected and websafe origins) iff PROTECTED_WEB.
+ if (policy->IsStorageProtected(origin.GetOrigin()) &&
+ BrowsingDataHelper::HasWebScheme(origin.GetOrigin()) &&
+ origin_set_mask & PROTECTED_WEB)
+ return true;
+
+ return false;
+}

Powered by Google App Engine
This is Rietveld 408576698