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

chrome/common/content_settings_pattern_unittest.cc

 // Copyright (c) 2012 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/common/content_settings_pattern.h"
 
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
@@ skipping 176 matching lines @@
   EXPECT_TRUE(ContentSettingsPattern::Wildcard().Matches(
       GURL("file:///foo/bar.txt")));
 
   EXPECT_STREQ("*", ContentSettingsPattern::Wildcard().ToString().c_str());
 
   EXPECT_EQ(ContentSettingsPattern::IDENTITY,
             ContentSettingsPattern::Wildcard().Compare(
                 ContentSettingsPattern::Wildcard()));
 }
 
-TEST(ContentSettingsPatternTest, TrimEndingDotFromHost) {
+TEST(ContentSettingsPatternTest, TrimEndingDots) {
   EXPECT_TRUE(Pattern("www.example.com").IsValid());
   EXPECT_TRUE(Pattern("www.example.com").Matches(
       GURL("http://www.example.com")));
   EXPECT_TRUE(Pattern("www.example.com").Matches(
-      GURL("http://www.example.com.")));
+      GURL("http://www.example.com..")));
 
-  EXPECT_TRUE(Pattern("www.example.com.").IsValid());
+  EXPECT_TRUE(Pattern("www.example.com..").IsValid());
   EXPECT_STREQ("www.example.com",
-               Pattern("www.example.com.").ToString().c_str());
+               Pattern("www.example.com..").ToString().c_str());
 
-  EXPECT_TRUE(Pattern("www.example.com.") == Pattern("www.example.com"));
+  EXPECT_TRUE(Pattern("www.example.com..") == Pattern("www.example.com"));
+}
+
+TEST(ContentSettingsPatternTest, TrimEndingDotsWithStringConsistsOfDots) {
+  EXPECT_TRUE(Pattern(".").IsValid());
+  EXPECT_TRUE(Pattern(".").Matches(GURL("http://.")));
+  EXPECT_TRUE(Pattern(".").Matches(GURL("http://..")));
+
+  EXPECT_TRUE(Pattern("..").IsValid());
+  EXPECT_STREQ(".", Pattern("..").ToString().c_str());
+
+  EXPECT_TRUE(Pattern("..") == Pattern("."));
 }
 
 TEST(ContentSettingsPatternTest, FromString_WithNoWildcards) {
   // HTTP patterns with default port.
   EXPECT_TRUE(Pattern("http://www.example.com:80").IsValid());
   EXPECT_STREQ("http://www.example.com:80",
                Pattern("http://www.example.com:80").ToString().c_str());
   // HTTP patterns with none default port.
   EXPECT_TRUE(Pattern("http://www.example.com:81").IsValid());
   EXPECT_STREQ("http://www.example.com:81",
@@ skipping 438 matching lines @@
   // file:/// normalization.
   EXPECT_STREQ("file:///tmp/test.html",
                Pattern("file:///tmp/bar/../test.html").ToString().c_str());
 
   // Invalid patterns.
   EXPECT_STREQ("", Pattern("*example.com").ToString().c_str());
   EXPECT_STREQ("", Pattern("example.*").ToString().c_str());
   EXPECT_STREQ("", Pattern("*\xC4\x87ira.com").ToString().c_str());
   EXPECT_STREQ("", Pattern("\xC4\x87ira.*").ToString().c_str());
 }