fix partially broken patterns, delete totally brocken patterns

chrome/browser/content_settings/content_settings_pref_provider_unittest.cc

 // Copyright (c) 2011 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/content_settings/content_settings_pref_provider.h"
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
 #include "chrome/browser/content_settings/content_settings_mock_observer.h"
 #include "chrome/browser/prefs/browser_prefs.h"
@@ skipping 471 matching lines @@
                                               &settings);
   ASSERT_TRUE(NULL != settings) << "Dictionary has no key: "
                                 << primary_pattern.ToString();
   ASSERT_EQ(1U, settings->size());
   settings->GetInteger("javascript", &setting_value);
   EXPECT_EQ(setting_value, CONTENT_SETTING_ALLOW);
 
   provider.ShutdownOnUIThread();
 }
 
+
+TEST_F(PrefProviderTest, FixOrRemoveMalformedPatternKeysFromObsoletePref) {
+  TestingProfile profile;
+  PrefService* prefs = profile.GetPrefs();
+
+  // Set obsolete preference for content settings pattern.
+  scoped_ptr<DictionaryValue> settings_dictionary(new DictionaryValue());
+  settings_dictionary->SetInteger("cookies", 2);
+  settings_dictionary->SetInteger("images", 2);
+  settings_dictionary->SetInteger("popups", 2);
+  scoped_ptr<DictionaryValue> all_settings_dictionary(new DictionaryValue());
+  // Good pattern key.
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.example.com", settings_dictionary->DeepCopy());
+  // Bad pattern key that will be ignored since there is already a good pattern
+  // key for the primary patter of the bad pattern key.
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.example.com,", settings_dictionary->DeepCopy());
+
+  // Bad pattern key that should be removed.
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.broken.com*", settings_dictionary->DeepCopy());
+
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.bar.com,", settings_dictionary->DeepCopy());
+
+  // Bad pattern key with a trailing comma that is supposed to be fixed.
+  // A trailing comma means that the secondary pattern string is empty and hence
+  // invalid.
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.foo.com,", settings_dictionary->DeepCopy());
+  // Bad pattern key with an invalid secondary pattern that should be removed.
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.foo.com,error*", settings_dictionary->DeepCopy());
+  // Pattern keys with valid pattern pairs.
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.foo.com,[*.]bar.com", settings_dictionary->DeepCopy());
+  prefs->Set(prefs::kContentSettingsPatterns, *all_settings_dictionary);
+  all_settings_dictionary->SetWithoutPathExpansion(
+      "http://www.example2.com,*", settings_dictionary->DeepCopy());
+  prefs->Set(prefs::kContentSettingsPatterns, *all_settings_dictionary);
+
+  content_settings::PrefProvider provider(prefs, false);
+
+  // Tests that the broken pattern keys got fixed or removed.
+  const DictionaryValue* patterns_dictionary =
+      prefs->GetDictionary(prefs::kContentSettingsPatterns);
+  EXPECT_EQ(4U, patterns_dictionary->size());
+  EXPECT_TRUE(patterns_dictionary->HasKey("http://www.example.com"));
+  EXPECT_TRUE(patterns_dictionary->HasKey("http://www.bar.com"));
+  EXPECT_TRUE(patterns_dictionary->HasKey("http://www.foo.com"));
+  EXPECT_TRUE(patterns_dictionary->HasKey("http://www.example2.com"));
+
+  // Broken pattern keys that should be removed
+  EXPECT_FALSE(patterns_dictionary->HasKey("http://www.bar.com,"));
+  EXPECT_FALSE(patterns_dictionary->HasKey("http://www.foo.com,"));
+  EXPECT_FALSE(patterns_dictionary->HasKey("http://www.foo.com,error*"));
+  EXPECT_FALSE(patterns_dictionary->HasKey(
+      "http://www.foo.com,[*.]bar.com"));
+  EXPECT_FALSE(patterns_dictionary->HasKey("http://www.example2.com,*"));
+
+  EXPECT_FALSE(patterns_dictionary->HasKey("http://www.broken.com*"));
+
+  provider.ShutdownOnUIThread();
+}
+
 }  // namespace content_settings