Fix b180514: Some exception entries cannot be deleted from a setting panel.

chrome/common/content_settings_pattern_unittest.cc

Index: chrome/common/content_settings_pattern_unittest.cc
diff --git a/chrome/common/content_settings_pattern_unittest.cc b/chrome/common/content_settings_pattern_unittest.cc
index bd793382e87cc3a6c8019962622f52605084be16..670ad9aa12fd602e4dd68f4eb5cf1918f7a32305 100644
--- a/chrome/common/content_settings_pattern_unittest.cc
+++ b/chrome/common/content_settings_pattern_unittest.cc
@@ -206,6 +206,9 @@ TEST(ContentSettingsPatternTest, TrimEndingDotFromHost) {
                Pattern("www.example.com.").ToString().c_str());
 
   EXPECT_TRUE(Pattern("www.example.com.") == Pattern("www.example.com"));
+
+  EXPECT_TRUE(Pattern(".").IsValid());

[markusheintz_, 2013/04/02 19:16:46]

An empty domain (host) does not make much sense for defining site specific
content settings exceptions since it does not match any particular domain.

Valid content settings patterns require the domain (host) to be non empty.

We should not allow this pattern.

[yhirano, 2013/04/02 19:54:54]

Done.

+  EXPECT_STREQ(".", Pattern(".").ToString().c_str());
 }
 
 TEST(ContentSettingsPatternTest, FromString_WithNoWildcards) {
@@ -443,6 +446,10 @@ TEST(ContentSettingsPatternTest, InvalidPatterns) {
   EXPECT_STREQ("", Pattern("file://").ToString().c_str());
   EXPECT_FALSE(Pattern("file:///foo/bar.html:8080").IsValid());
   EXPECT_STREQ("", Pattern("file:///foo/bar.html:8080").ToString().c_str());
+
+  // Host have consequtive ending dots
+  EXPECT_FALSE(Pattern("www.example.com..").IsValid());
+  EXPECT_FALSE(Pattern("..").IsValid());
 }
 
 TEST(ContentSettingsPatternTest, UnequalOperator) {