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

Unified Diff: chrome/browser/content_settings/content_settings_details.cc

Issue 440423003: Clean content_settings_pattern_parser.* from unnecessary dependencies. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed the clients Created 6 years, 4 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/content_settings/content_settings_details.cc
diff --git a/chrome/browser/content_settings/content_settings_details.cc b/chrome/browser/content_settings/content_settings_details.cc
index b6410e292ec76c82768d379f52319e3248719aee..348110de3b089cbc6ada97f775a7b754ab5e7068 100644
--- a/chrome/browser/content_settings/content_settings_details.cc
+++ b/chrome/browser/content_settings/content_settings_details.cc
@@ -4,6 +4,8 @@
#include "chrome/browser/content_settings/content_settings_details.h"
+#include "extensions/common/constants.h"
+
ContentSettingsDetails::ContentSettingsDetails(
const ContentSettingsPattern& primary_pattern,
const ContentSettingsPattern& secondary_pattern,
@@ -14,3 +16,8 @@ ContentSettingsDetails::ContentSettingsDetails(
type_(type),
resource_identifier_(resource_identifier) {
}
+
+bool ContentSettingsDetails::update_all() const {
+ return primary_pattern_.ToString(extensions::kExtensionScheme).empty() &&
+ secondary_pattern_.ToString(extensions::kExtensionScheme).empty();
+}

Powered by Google App Engine
This is Rietveld 408576698