Teaching BrowsingDataRemover how to delete application data.

chrome/browser/browsing_data_helper.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browsing_data_helper.h"
 
 #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"
 #include "googleurl/src/gurl.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
 
 // Static
 bool BrowsingDataHelper::IsValidScheme(const std::string& scheme) {
   // Special-case `file://` scheme iff cookies and site data are enabled via
   // the `--allow-file-cookies` CLI flag.
@@ skipping 14 matching lines @@
 
 // Static
 bool BrowsingDataHelper::IsValidScheme(const WebKit::WebString& scheme) {
   return BrowsingDataHelper::IsValidScheme(UTF16ToUTF8(scheme));
 }
 
 // Static
 bool BrowsingDataHelper::HasValidScheme(const GURL& origin) {
   return BrowsingDataHelper::IsValidScheme(origin.scheme());
 }
+
+// Static
+bool BrowsingDataHelper::DoesOriginMatchMask(const GURL& origin,
+    int origin_set_mask, ExtensionSpecialStoragePolicy* policy) {
+  // If a websafe origin is unprotected, it matches iff UNPROTECTED_WEB.
+  if (!policy->IsStorageProtected(origin.GetOrigin()) &&
+      BrowsingDataHelper::HasValidScheme(origin.GetOrigin()) &&
+      origin_set_mask & UNPROTECTED_WEB)
+    return true;
+
+  // Packaged apps and extensions match iff EXTENSION.
+  if (!BrowsingDataHelper::HasValidScheme(origin.GetOrigin()) &&

[Bernhard Bauer, 2012/05/31 15:57:31]

Could you change this check to just `origin.scheme() ==
chrome::kExtensionScheme`?  It feels kinda weird to assume that everything
that's not a valid scheme is an automatically an extension.

Alternatively, now that we do treat extensions as potentially clearable browsing
data, maybe we should change the IsValidScheme implementation?

[Mike West, 2012/06/01 13:35:17]

Running with `isWebScheme` and `isExtensionScheme` in
http://codereview.chromium.org/10454113/

+      origin_set_mask & EXTENSION)
+    return true;
+
+  // Hosted applications (protected and websafe origins) iff PROTECTED_WEB.
+  if (policy->IsStorageProtected(origin.GetOrigin()) &&
+      BrowsingDataHelper::HasValidScheme(origin.GetOrigin()) &&
+      origin_set_mask & PROTECTED_WEB)
+    return true;
+
+  return false;
+}