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

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

Issue 7399015: Sync Promo: Add a way to collapse the sync promo (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address review feedback Created 9 years, 5 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_pref_provider.cc
diff --git a/chrome/browser/content_settings/content_settings_pref_provider.cc b/chrome/browser/content_settings/content_settings_pref_provider.cc
index 56ac49b9fc5e60583aa9e405b6d9fb500ff1b71d..659f522d746da467d828f0104650eb55502de425 100644
--- a/chrome/browser/content_settings/content_settings_pref_provider.cc
+++ b/chrome/browser/content_settings/content_settings_pref_provider.cc
@@ -1031,6 +1031,8 @@ void PrefProvider::MigrateObsoleteContentSettingsPatternPref(
i != all_settings_dictionary->end_keys();
++i) {
const std::string& key(*i);
+ if (key.empty())
James Hawkins 2011/07/20 01:55:26 When does this happen?
sail 2011/07/20 19:40:25 Oops, this is a separate change. Removed.
+ continue;
// Validate pattern string and skip it if it is invalid.
std::pair<ContentSettingsPattern, ContentSettingsPattern> pattern_pair =

Powered by Google App Engine
This is Rietveld 408576698