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

Unified Diff: chrome/browser/content_settings/content_settings_utils.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_utils.cc
diff --git a/chrome/browser/content_settings/content_settings_utils.cc b/chrome/browser/content_settings/content_settings_utils.cc
index 5c27fdb414dde52bf3f314310a317bafcb4221ac..cf396df6e94a5035e5961324cbe1aa2da3682156 100644
--- a/chrome/browser/content_settings/content_settings_utils.cc
+++ b/chrome/browser/content_settings/content_settings_utils.cc
@@ -17,6 +17,7 @@
#include "chrome/browser/content_settings/host_content_settings_map.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/content_settings_pattern.h"
+#include "extensions/common/constants.h"
#include "url/gurl.h"
namespace {
@@ -79,9 +80,9 @@ bool GetTypeFromName(const std::string& name,
std::string CreatePatternString(
const ContentSettingsPattern& item_pattern,
const ContentSettingsPattern& top_level_frame_pattern) {
- return item_pattern.ToString()
+ return item_pattern.ToString(extensions::kExtensionScheme)
+ std::string(kPatternSeparator)
- + top_level_frame_pattern.ToString();
+ + top_level_frame_pattern.ToString(extensions::kExtensionScheme);
}
PatternPair ParsePatternString(const std::string& pattern_str) {
@@ -109,9 +110,11 @@ PatternPair ParsePatternString(const std::string& pattern_str) {
PatternPair pattern_pair;
pattern_pair.first =
- ContentSettingsPattern::FromString(pattern_str_list[0]);
+ ContentSettingsPattern::FromString(extensions::kExtensionScheme,
+ pattern_str_list[0]);
pattern_pair.second =
- ContentSettingsPattern::FromString(pattern_str_list[1]);
+ ContentSettingsPattern::FromString(extensions::kExtensionScheme,
+ pattern_str_list[1]);
return pattern_pair;
}

Powered by Google App Engine
This is Rietveld 408576698